fix: install CodeBoarding engine from PyPI#44
Merged
Conversation
Amp-Thread-ID: https://ampcode.com/threads/T-019edcb0-130a-713d-8e97-997ec5e29b33 Co-authored-by: Amp <amp@ampcode.com>
ivanmilevtues
force-pushed
the
fix/pip-package-action-build
branch
from
3efd5b6 to
4cac22f
Compare
Bump the codeboarding_version default to 0.12.3 (latest published) across action.yml and the README. Also switch the action's setup-python from 3.13 to 3.12: the codeboarding package's transitive dep langchain-cerebras (>=0.8) is published only for >=3.11,<3.13, so a 3.13 runner cannot resolve the engine install (true for both 0.12.2 and 0.12.3). Verified 0.12.3 installs cleanly on 3.12 and the codeboarding-setup entry point is present. Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
The review workflow dogfoods via `uses: ./`, but on issue_comment events actions/checkout defaults to the repo's default branch (the event isn't tied to a PR), so /codeboarding ran main's action code instead of the PR's. Pass an explicit ref (refs/pull/<n>/head) for issue_comment so the comment path exercises the PR under review too; pull_request events keep checkout's default (the PR merge ref), so they are unchanged. Takes effect once on main, since GitHub reads the issue_comment workflow definition from the default branch. Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
Architecture review · 1 component changedgraph LR
n_Analysis_Orchestrator["Analysis Orchestrator"]
n_Structural_Diffing_Engine["Structural Diffing Engine"]
n_Mermaid_Visualization_Engine["Mermaid Visualization Engine"]
n_UX_Integration_Layer["UX #38; Integration Layer"]
n_Analysis_Orchestrator -- "triggers health checks and status reporting" --> n_UX_Integration_Layer
n_Structural_Diffing_Engine -- "provides structural change data for diagram gen…" --> n_Mermaid_Visualization_Engine
n_Structural_Diffing_Engine -- "supplies change metrics and rendered diffs for…" --> n_UX_Integration_Layer
n_Mermaid_Visualization_Engine -- "queries specific file changes to resolve compon…" --> n_Structural_Diffing_Engine
n_Mermaid_Visualization_Engine -- "provides component change counts for summary re…" --> n_UX_Integration_Layer
n_UX_Integration_Layer -- "applies visibility filters to diagram rendering" --> n_Mermaid_Visualization_Engine
classDef added fill:#1f883d,stroke:#0b5d23,color:#ffffff;
classDef modified fill:#bf8700,stroke:#7d4e00,color:#ffffff;
classDef deleted fill:#cf222e,stroke:#82071e,color:#ffffff,stroke-dasharray:5 3;
class n_Analysis_Orchestrator,n_UX_Integration_Layer modified;
Colors indicate component changes compared to target branch Download the PR analysis artifacts from this workflow artifact. Analysis Orchestrator : 1 changed sub-component, 1 file changed
UX & Integration Layer : 1 file changed
Explore this PR’s architecture in your browser or VS Code. codeboarding-action · run 27796755068 |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
Validation
Self-test note
This repo's pull_request review workflow checks out the PR and runs the action with uses: ./, so the PR-open workflow should exercise this branch's action code. Slash-command issue_comment reruns may use the default branch checkout unless that workflow is changed to fetch the PR head explicitly.