Skip to content

Clarify Codex worktree project resolution - #39

Open
jskoiz wants to merge 1 commit into
mainfrom
jerry/avm-42-clarify-codex-worktree-project-resolution-behavior
Open

Clarify Codex worktree project resolution#39
jskoiz wants to merge 1 commit into
mainfrom
jerry/avm-42-clarify-codex-worktree-project-resolution-behavior

Conversation

@jskoiz

@jskoiz jskoiz commented May 6, 2026

Copy link
Copy Markdown
Owner

Closes AVM-42

Supersedes draft PR #28, which could not be marked ready for review because the GitHub connector ready-for-review mutation repeatedly failed with GithubGraphQLAPIError: Field 'htmlUrl' doesn't exist on type 'PullRequest' and CLI fallback was unavailable due to invalid gh auth.

Summary

  • clarify in architecture docs that project resolution tagging is separate from AI tool scanning
  • document that resolved paths under .claude/worktrees, .codex/worktrees, and .git/worktrees are marked worktree-like

Validation

  • CLANG_MODULE_CACHE_PATH="$PWD/.build/clang-module-cache" SWIFTPM_CACHE_PATH="$PWD/.build/swiftpm-cache" swift test --disable-sandbox --filter ProjectResolverTests
  • git diff --check

Full swift test --disable-sandbox was attempted previously and reached the suite, but this sandbox blocks the live listener integration with listen EPERM on 127.0.0.1:39123.

@chatgpt-codex-connector

Copy link
Copy Markdown

You have reached your Codex usage limits for code reviews. You can see your limits in the Codex usage dashboard.
To continue using code reviews, add credits to your account and enable them for code reviews in your settings.

@greptile-apps greptile-apps 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.

Your free trial has ended. If you'd like to continue receiving code reviews, you can add a payment method here.

@jskoiz

jskoiz commented May 7, 2026

Copy link
Copy Markdown
Owner Author

Codex review pass: I did not find a blocking issue in this pass. Caveat: I reviewed the PR diff and reported check/validation metadata via gh only; I did not run the Swift test suite locally.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant