Preserve extension command contracts and recovery fidelity

.agents/pm/extensions/.managed-extensions.json

@@ -1,26 +1,26 @@
 {
   "version": 1,
-  "updated_at": "2026-07-10T21:12:15.248Z",
+  "updated_at": "2026-07-11T07:56:38.083Z",
   "entries": [
     {
       "name": "pm-changelog",
       "directory": "pm-changelog",
       "scope": "project",
-      "manifest_version": "2026.7.10",
+      "manifest_version": "2026.7.11",
       "manifest_entry": "./dist/extension.js",
       "capabilities": [
         "commands",
         "importers",
         "schema"
       ],
       "installed_at": "2026-05-23T20:26:03.509Z",
-      "updated_at": "2026-07-10T11:32:53.862Z",
+      "updated_at": "2026-07-11T05:52:32.969Z",
       "source": {
         "kind": "npm",
         "input": "npm:pm-changelog",
         "location": "package",
         "package": "pm-changelog",
-        "version": "2026.7.10"
+        "version": "2026.7.11"
       }
     }
   ]

.agents/pm/history/pm-o71e.jsonl

@@ -1 +1,21 @@
 {"ts":"2026-07-11T02:49:52.213Z","author":"maintainer-agent","op":"create","patch":[{"op":"add","path":"/metadata/id","value":"pm-o71e"},{"op":"add","path":"/metadata/title","value":"GH-505: nested extension failures suggest irrelevant missing flags instead of preserving tracker recovery"},{"op":"add","path":"/metadata/description","value":"When an extension command internally invokes another pm command and the nested command fails with a typed error (observed: tracker_not_initialized from 'list-all --limit 10000' inside pm-todos import against an uninitialized --pm-path), the outer command_failed envelope invents an unrelated missing option from the NESTED invocation: missing=['--limit'] + suggested_retry appending '--limit <value>' to the user's todos-import command line. --limit is not an importer flag and cannot fix the failure; the correct recovery (init or fix --pm-path) was already present in the nested error and gets discarded. Agents follow structured suggested_retry verbatim — this sends them toward an invalid command instead of the storage fix. Expected: preserve the nested typed error code/recovery when wrapping extension failures; never infer top-level missing flags from a nested command's fields; omit missing/suggested_retry unless the option belongs to the user-invoked command's contract."},{"op":"add","path":"/metadata/type","value":"Issue"},{"op":"add","path":"/metadata/status","value":"open"},{"op":"add","path":"/metadata/priority","value":2},{"op":"add","path":"/metadata/tags","value":["agent-ux","cli","error-recovery","extensions","gh-issue"]},{"op":"add","path":"/metadata/created_at","value":"2026-07-11T02:49:52.213Z"},{"op":"add","path":"/metadata/updated_at","value":"2026-07-11T02:49:52.213Z"},{"op":"add","path":"/metadata/author","value":"maintainer-agent"},{"op":"add","path":"/metadata/estimated_minutes","value":90},{"op":"add","path":"/metadata/acceptance_criteria","value":"Wrapped extension failures preserve the nested typed error code and recovery verbatim; missing/suggested_retry only ever reference options in the user-invoked command's contract; repro (todos import against uninitialized --pm-path) yields tracker_not_initialized recovery, not --limit; regression test covers nested-failure wrapping; GH-505 closed with a comment linking the fix"},{"op":"add","path":"/metadata/risk","value":"medium"},{"op":"add","path":"/metadata/confidence","value":"high"},{"op":"add","path":"/metadata/dependencies","value":[{"id":"pm-gdi7","kind":"related","created_at":"2026-07-11T02:49:52.213Z"}]}],"before_hash":"3cc22dff72be7b14824654a7a64ea62b04799939b2fee54c1b5f52ca60bf6df0","after_hash":"126bb169dc8026e4515d63ec2095b455f8663044347071a1f1def0f03ff9b2e4","message":""}
+{"ts":"2026-07-11T04:45:42.056Z","author":"codex-root","op":"claim","patch":[{"op":"replace","path":"/metadata/updated_at","value":"2026-07-11T04:45:42.056Z"},{"op":"add","path":"/metadata/assignee","value":"codex-root"}],"before_hash":"126bb169dc8026e4515d63ec2095b455f8663044347071a1f1def0f03ff9b2e4","after_hash":"35cb6d62f7d1192c9236663ab13ee8c24fb87503c22e55f6ddfe238a425efd40"}
+{"ts":"2026-07-11T04:45:42.878Z","author":"codex-root","op":"update","patch":[{"op":"replace","path":"/metadata/updated_at","value":"2026-07-11T04:45:42.878Z"},{"op":"replace","path":"/metadata/status","value":"in_progress"}],"before_hash":"35cb6d62f7d1192c9236663ab13ee8c24fb87503c22e55f6ddfe238a425efd40","after_hash":"5817b1f834236cc6772024711ba4346903c9b4cbd672844d472b23f01ae0bf76","message":"Start broad extension contract fidelity and recovery implementation bundle"}
+{"ts":"2026-07-11T04:45:43.644Z","author":"codex-root","op":"comment_add","patch":[{"op":"replace","path":"/metadata/updated_at","value":"2026-07-11T04:45:43.644Z"},{"op":"add","path":"/metadata/comments","value":[{"created_at":"2026-07-11T04:45:43.644Z","author":"codex-root","text":"Implementation start: bundled with GH-503/504/505/507 because flattened aliases, bare importer/exporter contracts, nested typed recovery, and boolean retry arity share extension registration and structured-error surfaces. Duplicate check used pm search --status all; closed prior extension SDK work remains closed."}]}],"before_hash":"5817b1f834236cc6772024711ba4346903c9b4cbd672844d472b23f01ae0bf76","after_hash":"c5cd51cb8e7ad2520685ec2642ece6cc0936748b55c94e29d473a5d268fcbc99"}
+{"ts":"2026-07-11T04:48:28.831Z","author":"codex-root","op":"comment_add","patch":[{"op":"add","path":"/metadata/comments/1","value":{"created_at":"2026-07-11T04:48:28.830Z","author":"codex-root","text":"Pause note: root cause investigation reached structured recovery assembly in src/cli/main.ts, but no verified patch is recorded yet. Item remains open and unclaimed."}},{"op":"replace","path":"/metadata/updated_at","value":"2026-07-11T04:48:28.831Z"}],"before_hash":"c5cd51cb8e7ad2520685ec2642ece6cc0936748b55c94e29d473a5d268fcbc99","after_hash":"1554213eee77ce3630ffcad5343f6f6b9e9098010b021544700bf1c24937658d"}
+{"ts":"2026-07-11T04:48:34.615Z","author":"codex-root","op":"update","patch":[{"op":"replace","path":"/metadata/updated_at","value":"2026-07-11T04:48:34.615Z"},{"op":"replace","path":"/metadata/status","value":"open"}],"before_hash":"1554213eee77ce3630ffcad5343f6f6b9e9098010b021544700bf1c24937658d","after_hash":"a822bc65b0928aa9bc68e0ae6d3f91996f30c5b55f8f5236835fc93b3b4cb84d","message":"Pause incomplete implementation; return item to open"}
+{"ts":"2026-07-11T04:48:44.066Z","author":"codex-root","op":"release","patch":[{"op":"remove","path":"/metadata/assignee"},{"op":"replace","path":"/metadata/updated_at","value":"2026-07-11T04:48:44.066Z"}],"before_hash":"a822bc65b0928aa9bc68e0ae6d3f91996f30c5b55f8f5236835fc93b3b4cb84d","after_hash":"216ecc840ccff734c4bd1289a89e2e3cef1816c9d666a047ed8334b14e131a1f"}
+{"ts":"2026-07-11T04:53:45.562Z","author":"codex-root","op":"claim","patch":[{"op":"replace","path":"/metadata/updated_at","value":"2026-07-11T04:53:45.562Z"},{"op":"add","path":"/metadata/assignee","value":"codex-root"}],"before_hash":"216ecc840ccff734c4bd1289a89e2e3cef1816c9d666a047ed8334b14e131a1f","after_hash":"aa212c5142231a2fc19d72fcb6cfb2a8ad7ad0857e0ef862c8a8beb83c6abc9b"}
+{"ts":"2026-07-11T04:53:46.294Z","author":"codex-root","op":"update","patch":[{"op":"replace","path":"/metadata/updated_at","value":"2026-07-11T04:53:46.294Z"},{"op":"replace","path":"/metadata/status","value":"in_progress"}],"before_hash":"aa212c5142231a2fc19d72fcb6cfb2a8ad7ad0857e0ef862c8a8beb83c6abc9b","after_hash":"c4958b6c23e71d649af5ec7670c5cad07051271b235c317ab24f0485a9cc0d4d","message":"Resume broad implementation after recoverable runner interruption"}
+{"ts":"2026-07-11T04:58:45.610Z","author":"codex-root","op":"files_add","patch":[{"op":"replace","path":"/metadata/updated_at","value":"2026-07-11T04:58:45.610Z"},{"op":"add","path":"/metadata/files","value":[{"path":"src/cli/main.ts","scope":"project","note":"extension-aware structured recovery fidelity"}]}],"before_hash":"c4958b6c23e71d649af5ec7670c5cad07051271b235c317ab24f0485a9cc0d4d","after_hash":"4444b9a91f6aae611f16febd48b6315a19b751c74354dfaffee88aa6e799ee35"}
+{"ts":"2026-07-11T04:58:46.528Z","author":"codex-root","op":"tests_add","patch":[{"op":"replace","path":"/metadata/updated_at","value":"2026-07-11T04:58:46.528Z"},{"op":"add","path":"/metadata/tests","value":[{"command":"node scripts/run-tests.mjs test -- tests/unit/cli/cli-main-errors.spec.ts tests/unit/extensions/extension-loader.spec.ts","scope":"project","timeout_seconds":240}]}],"before_hash":"4444b9a91f6aae611f16febd48b6315a19b751c74354dfaffee88aa6e799ee35","after_hash":"401e90d8368a0d2c7189b3f1b9dce5a6395f6ccac23720b56ae2b3df42125012"}
+{"ts":"2026-07-11T04:58:51.007Z","author":"codex-root","op":"files_add","patch":[{"op":"add","path":"/metadata/files/1","value":{"path":"tests/unit/cli/cli-main-errors.spec.ts","scope":"project","note":"nested extension recovery regression coverage"}},{"op":"replace","path":"/metadata/updated_at","value":"2026-07-11T04:58:51.007Z"}],"before_hash":"401e90d8368a0d2c7189b3f1b9dce5a6395f6ccac23720b56ae2b3df42125012","after_hash":"1ac740091f59a92fcd6b503ba0b94156423fd9d27073118bd824e90f87352b71"}
+{"ts":"2026-07-11T05:05:09.028Z","author":"codex-root","op":"comment_add","patch":[{"op":"add","path":"/metadata/comments/2","value":{"created_at":"2026-07-11T05:05:09.028Z","author":"codex-root","text":"Implementation checkpoint: contract and recovery patches are in the working tree; pnpm typecheck, pnpm lint, and focused CLI/extension suites pass. The first full coverage run was invalidated by a shared-state test and stopped after the test was rewritten as a pure helper assertion; full 100% coverage, temp installed-package proof, Greptile, commit/PR, and review loops remain required. Returning to open and releasing claim because work is paused."}},{"op":"replace","path":"/metadata/updated_at","value":"2026-07-11T05:05:09.028Z"}],"before_hash":"1ac740091f59a92fcd6b503ba0b94156423fd9d27073118bd824e90f87352b71","after_hash":"7b6d77f64be4a45990c1dad840eaaa0b83a1bc492bb2d58f3701f8befc19a76e"}
+{"ts":"2026-07-11T05:05:10.064Z","author":"codex-root","op":"update","patch":[{"op":"replace","path":"/metadata/updated_at","value":"2026-07-11T05:05:10.064Z"},{"op":"replace","path":"/metadata/status","value":"open"}],"before_hash":"7b6d77f64be4a45990c1dad840eaaa0b83a1bc492bb2d58f3701f8befc19a76e","after_hash":"78bf7c0ad06b9af0745ba8fbc19b92cff2792eecb92d673e4b9d801f4aa54cd5","message":"Pause after verified focused implementation; full shipping gates remain"}
+{"ts":"2026-07-11T05:05:11.007Z","author":"codex-root","op":"release","patch":[{"op":"remove","path":"/metadata/assignee"},{"op":"replace","path":"/metadata/updated_at","value":"2026-07-11T05:05:11.007Z"}],"before_hash":"78bf7c0ad06b9af0745ba8fbc19b92cff2792eecb92d673e4b9d801f4aa54cd5","after_hash":"1a870a1280bca6f385567188d8646fb03b933751451d638a59329ee8520d30fb"}
+{"ts":"2026-07-11T05:06:12.727Z","author":"codex-root","op":"claim","patch":[{"op":"replace","path":"/metadata/updated_at","value":"2026-07-11T05:06:12.727Z"},{"op":"add","path":"/metadata/assignee","value":"codex-root"}],"before_hash":"1a870a1280bca6f385567188d8646fb03b933751451d638a59329ee8520d30fb","after_hash":"caf9bd12da4442553a16b4bb48a73781450ce3a409941e9074f4d98d9d97e6a9"}
+{"ts":"2026-07-11T05:06:13.469Z","author":"codex-root","op":"update","patch":[{"op":"replace","path":"/metadata/updated_at","value":"2026-07-11T05:06:13.469Z"},{"op":"replace","path":"/metadata/status","value":"in_progress"}],"before_hash":"caf9bd12da4442553a16b4bb48a73781450ce3a409941e9074f4d98d9d97e6a9","after_hash":"ed8d8513286b5b4ca7ef2975c4dca0330f60b644919ddb655fe0b99ff5681362","message":"Resume implementation through full verification, PR review convergence, and merge"}
+{"ts":"2026-07-11T05:52:23.737Z","author":"codex-root","op":"comment_add","patch":[{"op":"add","path":"/metadata/comments/3","value":{"created_at":"2026-07-11T05:52:23.737Z","author":"codex-root","text":"Verification evidence: pnpm lint passed (ESLint, zero jscpd clones, static quality and token-budget gates); full coverage passed 5532/5532 with exact 100% statements/branches/functions/lines. Fresh installed-package proof packed @unbrained/pm-cli@2026.7.10, installed it into /tmp/pm-extension-contract-proof, initialized a tracker, installed a TypeScript importer/exporter extension with omitted options, verified generated [file] help, successful positional import, healthy activation, and synthesized command contracts in extension describe."}},{"op":"replace","path":"/metadata/updated_at","value":"2026-07-11T05:52:23.737Z"}],"before_hash":"ed8d8513286b5b4ca7ef2975c4dca0330f60b644919ddb655fe0b99ff5681362","after_hash":"48ba7e2a7f082e1db57a9d381067ec6f2d60802b7344b451da22ae95c943016e"}
+{"ts":"2026-07-11T08:33:26.297Z","author":"codex-root","op":"comment_add","patch":[{"op":"add","path":"/metadata/comments/4","value":{"created_at":"2026-07-11T08:33:26.297Z","author":"codex-root","text":"Final evidence: nested extension command failures no longer synthesize irrelevant outer missing flags or retries, preserving the actionable nested structured recovery. Focused suite 231/231; full suite 5533/5533; exact coverage 100/100/100/100; lint and typecheck passed."}},{"op":"replace","path":"/metadata/updated_at","value":"2026-07-11T08:33:26.297Z"}],"before_hash":"48ba7e2a7f082e1db57a9d381067ec6f2d60802b7344b451da22ae95c943016e","after_hash":"d4cc2e52c6cf2d9f0bfd6e4c1021d022c7cde14af401a8fd7914ca7d0eab65c5"}
+{"ts":"2026-07-11T08:33:30.807Z","author":"codex-root","op":"close","patch":[{"op":"remove","path":"/metadata/assignee"},{"op":"replace","path":"/metadata/updated_at","value":"2026-07-11T08:33:30.807Z"},{"op":"replace","path":"/metadata/status","value":"closed"},{"op":"add","path":"/metadata/closed_at","value":"2026-07-11T08:33:30.806Z"},{"op":"add","path":"/metadata/resolution","value":"Outer extension recovery suppresses inferred retries when a nested structured failure owns recovery."},{"op":"add","path":"/metadata/expected_result","value":"Nested tracker recovery remains actionable and unrelated flags are never suggested."},{"op":"add","path":"/metadata/actual_result","value":"Nested failures retain their recovery path without bogus outer missing flags or suggested retries."},{"op":"add","path":"/metadata/close_reason","value":"Implemented nested extension recovery preservation with regression coverage."}],"before_hash":"d4cc2e52c6cf2d9f0bfd6e4c1021d022c7cde14af401a8fd7914ca7d0eab65c5","after_hash":"e9d5401bde9c38901cef3e261693b2bba5a6d691a601e2beffb32397e3daca8c"}
+{"ts":"2026-07-11T08:33:33.906Z","author":"codex-root","op":"release","patch":[{"op":"replace","path":"/metadata/updated_at","value":"2026-07-11T08:33:33.906Z"}],"before_hash":"e9d5401bde9c38901cef3e261693b2bba5a6d691a601e2beffb32397e3daca8c","after_hash":"8cd904a8785c3dd70a968d7a7d5afab8e8eccdf62f01e7d8355e591768732bfe"}