golem15/msd-core
4
0
Fork 0
Code Issues 1 Pull Requests Actions 2 Packages Projects Releases Wiki Activity

Merge pull request #3505 from gsd-build/fix/3346-codex-aot-toml-event-key

Browse Source 
fix(install): emit event-name leaf key for Codex AoT hooks migration (#3346)
This commit is contained in:
Tom Boucher
2026-05-14 14:03:54 -04:00
committed by GitHub
parent 0e1de9627a 434e6aa162
commit 3f054b4a96
3 changed files with 139 additions and 4 deletions
Download Patch File Download Diff File Expand all files Collapse all files

5
.changeset/3346-codex-aot-toml-key.md Normal file
View File

@@ -0,0 +1,5 @@
---
type: Fixed
pr: 3346
---
**Codex AoT hooks migration uses event-name leaf key, not location tuple** — `migrateCodexHooksMapFormat` in `bin/install.js` re-emitted the legacy `[hooks.<X>]` section's path segment verbatim as the leaf TOML key of the new `[[hooks.<EVENT>]]` two-level nested AoT block. When the legacy table key was a `<file>:<event>:<line>:<col>` location identifier (with the actual event name in an `event = "..."` body field), the migration produced a header like `[[hooks."C:\Users\helen\.codex\config.toml:session_start:0:0"]]`, which Codex 0.124.0+ refuses to load. The map-format and stale-namespaced-AoT branches now mirror the flat-AoT branch: when the section body declares `event = "..."`, that name wins as the leaf key (and `event` is excluded from the re-emitted handler body). `npx get-shit-done-cc@latest` no longer aborts the Codex runtime install on Windows configs that pre-date the AoT migration. (#3346)