Skip to content

fix(projection): ignore Claude runtime trace churn - #103

Merged
Ancienttwo merged 1 commit into
mainfrom
codex/ignore-claude-runtime-trace
Aug 10, 2026
Merged

fix(projection): ignore Claude runtime trace churn#103
Ancienttwo merged 1 commit into
mainfrom
codex/ignore-claude-runtime-trace

Conversation

@Ancienttwo

Copy link
Copy Markdown
Owner

Summary

  • exclude only .claude/.session-id and .claude/.trace.jsonl from worktree identity
  • keep .claude/settings.json digest-significant
  • align the 0.4.2 release surfaces and regression coverage

Root 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)
  • candidate archctx@0.4.2 tarball projection against the real repo-harness worktree crossed CodeGraph sync without the precondition failure
  • Claude review intentionally waived per user instruction

@Ancienttwo
Ancienttwo force-pushed the codex/ignore-claude-runtime-trace branch from e8fe53e to 35b7897 Compare August 10, 2026 03:42

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

💡 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"

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

P1 Badge 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";

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

P2 Badge 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 👍 / 👎.

@Ancienttwo
Ancienttwo merged commit a124462 into main Aug 10, 2026
8 checks passed
@Ancienttwo
Ancienttwo deleted the codex/ignore-claude-runtime-trace branch August 10, 2026 03:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant