claude-ops inventory: replace the regex bundle reader with a JavaScript parser

[kyle-sexton]

Problem

plugins/claude-ops/skills/inventory/scripts/inventory.py resolves built-in tool descriptions from the Claude Code bundle with regexes and a brace map. Review of #5619 ran about 20 rounds, each raising a new JavaScript-semantics corner (hoisted var, shadowing, nested functions, comments, template masking, number rendering, import matching). On 2.1.285 no displayed description changed after the first round.

The reader is now fail-closed by policy: a corner may leave a value unresolved, never produce a wrong one, and unresolved names reach drift intake through native_drift.py.

Proposal

If corner cases keep appearing, replace the regex reader with a real parser (for example acorn or esbuild run through node) and resolve bindings from its scope analysis.

Open question

Is node acceptable as a runtime dependency of the inventory skill? Today it needs only Python. The answer decides between a node-based parser and a pure-Python one.

Related

• fix(claude-ops): resolve built-in descriptions the inventory left unresolved #5619 (review history)

[kyle-sexton]

Two wrong-value findings from Codex's review of #5619's final head (9be6b2a), recorded here instead of fixed because that PR reached its last review round:

1. An imported binding wins over a nested lexical declaration (inventory.py _declaration, imported-name branch). With xx imported as "WRONG", function ff(){let xx="LOCAL";return xx} extracts WRONG; JavaScript returns LOCAL. Fix: check the reader's scope for a visible declaration before resolving through the import. Review thread on fix(claude-ops): resolve built-in descriptions the inventory left unresolved #5619: PRRT_kwDOTCGFQM6nyxVn
2. for-head bindings do not shadow a bound parameter (_scan block recursion, which adds only catch parameters to shadow). function ff(x){for(const x of ["LOCAL"]){return x}} called as ff("REAL") extracts REAL. Fix: add loop-head let/const names to the scope used to scan the loop body, as runtime values. Review thread on fix(claude-ops): resolve built-in descriptions the inventory left unresolved #5619: PRRT_kwDOTCGFQM6nyxVp

A scope-aware parser would close both at once.