docs(map-wayfind): document wayfind_status with --slug, not a positional - #408
Conversation
SKILL.md told the operator to run `wayfind_status <slug>` in two places, but
the CLI declares the argument as `--slug`, so following the skill verbatim
failed with:
wayfind_runner.py: error: unrecognized arguments: <slug>
Hit while charting a map: the chart step's closing status call and the
handoff-eligibility check both use this form. wayfind-reference.md already
documented `wayfind_status [--slug <slug>]` correctly, so only the skill body
was wrong.
Checked the rest of the wayfind commands documented in SKILL.md and
wayfind-reference.md against their argparse definitions — create_wayfind_map,
add_ticket, wire_blocking, claim_ticket, record_human_input, resolve_ticket,
add_fog, graduate_fog, rule_out_of_scope and emit_wayfind_handoff all match.
This was the only mismatch.
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: Organization UI Review profile: CHILL Plan: Pro Plus Run ID: 📒 Files selected for processing (3)
📝 WalkthroughWalkthroughThe PR updates two ChangesWayfind status syntax
Estimated code review effort: 1 (Trivial) | ~2 minutes Possibly related PRs
Poem
🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✨ Finishing Touches🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
…2.5 (#411) * fix(ci): allow packaging>=26 so twine check accepts Metadata-Version 2.5 The packaging>=24.2,<26 cap (#195) downgrades packaging to 25.0 in the build job, and 25.0 does not recognize Metadata-Version 2.5 emitted by current setuptools. twine check then fails with InvalidDistribution: '2.5' is not a valid metadata version. Verified locally: same wheel fails twine check with packaging<26 and passes with packaging>=26. * docs(changelog): document all unreleased changes since v3.24.1 Gate 12 of the release workflow found 15 commits since v3.24.1 but only 4 CHANGELOG entries. Adds entries for #379, #390, #393, #395, #396, #397, #399, #400/#401, #404, #408, #409 and the untracked 6-bug sweep (db028b6). * docs(spike): add specula applicability spike --------- Co-authored-by: Mikhail [azalio] Petrov <azalio@yandex-team.ru>
The
/map-wayfindskill body told the operator to runwayfind_status <slug>in two places, but the CLI declares that argument as--slug. Following the skill verbatim fails:Hit while actually charting a map — both affected lines are on the happy path: the
chartstep that shows the map after creation, and the handoff-eligibility check inhandoffmode.wayfind-reference.mdalready documentedwayfind_status [--slug <slug>]correctly, so only the always-loaded skill body was wrong — which is the copy an operator follows.Scope check
Compared every wayfind command documented in
SKILL.mdandwayfind-reference.mdagainst its argparse definition inwayfind_runner.py.jinja:create_wayfind_map,add_ticket,wire_blocking,claim_ticket,record_human_input,resolve_ticket,add_fog,graduate_fog,rule_out_of_scope,emit_wayfind_handoffall match their parsers.wayfind_statuswas the only mismatch.Verification
Edited
templates_src/**/*.jinjaand re-rendered;make check-renderreports the generated trees match.Summary by CodeRabbit
wayfind_statuscommand examples to use the explicit--slug <slug>option.