fix(projection): ignore Claude runtime trace churn - #103
Conversation
e8fe53e to
35b7897
Compare
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: e8fe53ea6a
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
| description: ArchContext runtime version expected by this action release. | ||
| required: false | ||
| default: "0.4.1" | ||
| default: "0.4.2" |
There was a problem hiding this comment.
Update the reusable workflow default to 0.4.2
When a caller omits the optional runtime_version input, .github/workflows/archcontext-organization-runner.yml lines 26-30 still supplies 0.4.1 to the action at line 72. This action now expects 0.4.2, and dist/review-action.mjs lines 62-64 rejects the mismatch before execution, so the reusable workflow's advertised default path will fail with runtime-version-mismatch; update that workflow default with the other release surfaces.
Useful? React with 👍 / 👎.
| @@ -1,5 +1,5 @@ | |||
| export const ARCHCONTEXT_PRODUCT_NAME = "archctx"; | |||
| export const ARCHCONTEXT_PRODUCT_VERSION = "0.4.1"; | |||
| export const ARCHCONTEXT_PRODUCT_VERSION = "0.4.2"; | |||
There was a problem hiding this comment.
Regenerate the version-bound no-LLM readbacks
Changing this constant also changes REVIEW_ACTION_NO_LLM_MODEL_DIGEST, because packages/cloud/runner/src/index.ts includes ARCHCONTEXT_PRODUCT_VERSION in that digest. The committed fg4-deterministic-conclusion-readback.json and fg6-no-provider-deterministic-readback.json still contain the 0.4.1 digest, so their documented inspect commands now fail with model digest must be no-provider digest; regenerate both artifacts for 0.4.2.
Useful? React with 👍 / 👎.
Summary
.claude/.session-idand.claude/.trace.jsonlfrom worktree identity.claude/settings.jsondigest-significantRoot cause
repo-harness records its live session while archctx performs the CodeGraph projection handshake. Those two operational files changed between the pre/post worktree digest, so every real projection failed with
architecture-docs-projection-worktree-changed-during-codegraph-sync.Verification
bun run verify(1219 tests, full verification PASS)archctx@0.4.2tarball projection against the real repo-harness worktree crossed CodeGraph sync without the precondition failure