[codex] Fix Claude setup and macOS path issues#599
Closed
Ununp3ntium115 wants to merge 1 commit into
Closed
Conversation
Fixes Claude integration issues around codegraph_files auto-allow, installer backups, codegraph_context guidance, and ArkTS .ets detection. Fixes macOS-related path issues around non-ASCII git paths and symlink escapes, and documents the macOS FTS5 source-build workaround. Refs colbymchenry#565 colbymchenry#486 colbymchenry#571 colbymchenry#512 colbymchenry#541 colbymchenry#527 colbymchenry#305.
Author
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
1 participant
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
codegraph_filesto Claude auto-allow permissions, create non-destructive installer backups, and makecodegraph_contextguidance/schema keyword-oriented..etsfiles as TypeScript for first-pass indexing.Root Cause
Claude-facing setup had drifted from the current MCP tool surface, the installer could overwrite existing agent config without a reliable restore point, and
codegraph_contextwording encouraged agents to send verbose natural-language tasks into a keyword-backed lookup.For the macOS/path issues, Git quotes non-ASCII paths by default, so tracked files under CJK directories could reach extension detection as quoted octal escapes. Path containment also only used lexical
path.resolve, so a symlink inside the project could resolve outside the project root after passing validation.Issues
Fixes #565
Fixes #486
Fixes #571
Fixes #512
Fixes #541
Fixes #527
Fixes #305
Validation
npx -y -p node@22 node ./node_modules/vitest/vitest.mjs run(52 files, 1102 passed, 2 skipped)npm test -- __tests__/extraction.test.tsnpm test -- __tests__/security.test.ts __tests__/installer.test.ts __tests__/installer-targets.test.ts __tests__/mcp-tool-allowlist.test.tsnpm run buildgit diff --check