Skip to content

Fix generated Codex skill paths - #2115

Closed
msr-hickory wants to merge 1 commit into
garrytan:mainfrom
msr-hickory:codex/fix-gstack-generated-codex-paths
Closed

msr-hickory wants to merge 1 commit into
garrytan:mainfrom
msr-hickory:codex/fix-gstack-generated-codex-paths

Conversation

@msr-hickory

Copy link
Copy Markdown
Contributor

Summary

  • switch generated Codex skill references from ~/.Codex/.Codex to lowercase ~/.codex/.codex paths for case-sensitive hosts
  • restore intended Claude/Codex wording in generated cross-model review guidance

Verification

  • rg -n --fixed-strings '~/.Codex' .agents/skills/gstack returned no matches from ai-config checkout
  • rg -n --fixed-strings '.Codex/' .agents/skills/gstack returned no matches from ai-config checkout
  • rg -n 'Codex and Codex|both Codex and Codex|Codex \+ Codex|Codex adversarial subagent' .agents/skills/gstack returned no matches from ai-config checkout

Use the lowercase Codex skill home on case-sensitive hosts and preserve Claude/Codex wording in generated cross-model review guidance.
@trunk-io

trunk-io Bot commented Jun 26, 2026

Copy link
Copy Markdown

Merging to main in this repository is managed by Trunk.

  • To merge this pull request, check the box to the left or comment /trunk merge below.

After your PR is submitted to the merge queue, this comment will be automatically updated with its status. If the PR fails, failure details will also be posted here

@time-attack

Copy link
Copy Markdown
Contributor

@16francej — recommend closing this PR as superseded by #1996. This branch changes generated outputs, removes a source template, and does not update generator source or behavior coverage; #1996 is the narrow source-level fix for the project-instruction path problem. Preserve the filename report and contributor attribution in #1996, while excluding global substitutions, unsupported paths, and template deletion.

@capy-ai

capy-ai Bot commented Oct 3, 2026

Copy link
Copy Markdown
Contributor

Thanks for the PR, @msr-hickory. Closing this one: it edits generated SKILL.md output rather than the generator, and the source-level fix is #1996's CLAUDE.md→AGENTS.md path rewrite, already on main. Appreciate you taking the time.

@capy-ai capy-ai Bot closed this Oct 3, 2026
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.

2 participants