[alexsoyes]

🎯 What & why

Codex displays AIDD hooks as “Unnamed hook”, making review and trust decisions difficult. Name the existing handlers and teach the hook generator to name future Claude and Codex handlers.

🛠️ How it works

statusMessage identifies each handler in Codex's hook review. The generator requires and validates a descriptive message for Claude and Codex without adding unsupported fields to other tool formats.

🧪 How to verify

• In an isolated Codex 0.166 session, review the plugin hooks: Context, Telemetry and UI handlers display their names.
• Generate a Claude and Codex hook with 08-hook-generate: both include statusMessage and existing hooks remain intact. Exercised in an isolated Codex session.
• Native Copilot and Claude sessions and real OpenCode/Kilo CLI runs executed the named memory hook; flat builds for Copilot, Cursor, OpenCode and Kilo remained byte-identical. Cursor native runtime could not be verified without authentication.
• node --test scripts/__tests__/aidd-telemetry-hooks.test.js scripts/__tests__/aidd-ui-validate.test.js: 120 tests passed.

⚠️ Heads-up

Changing a hook definition requires reviewing its trust again. The maintenance plugin is not yet on next; its hook is updated separately in #1145.

✅ I certify

• I DO CERTIFY I READ EACH LINE OF THE PULL REQUEST BECAUSE I AM A SOFTWARE ENGINEER, NOT A AI PUPPY.

[blafourcade]

Bien vu, Codex en a besoin.

Conflit : #1146 remplace update_memory.js par memory.cjs, #1159 réécrit l'entrée aidd-ui. La seconde mergée reprend le statusMessage.

[alexsoyes]

@blafourcade j'adresse tes changements et on peut merge je pense, de toute façon on va devoir refaire le hook ne serait-ce que pour formater les descriptions :)

[alexsoyes]

Revue OpenCode

Commit examiné : 23ed08194213ca15dc4b93d33cd240d907195401 ; la PR peut avoir évolué depuis.

Lot 1

Verdict: approve. No actionable defects found in the supplied changes.

The hook definitions name each existing handler, and the generator instructions require and validate names for new Claude Code and Codex handlers. Runtime behavior was not independently verified; the PR snapshot reports isolated-session testing, with Cursor native runtime unverified.

Verdict de ce lot : approve