Skip to content

docs(map-wayfind): document wayfind_status with --slug, not a positional - #408

Merged
azalio merged 1 commit into
mainfrom
fix-wayfind-status-slug-flag
Aug 10, 2026
Merged

docs(map-wayfind): document wayfind_status with --slug, not a positional#408
azalio merged 1 commit into
mainfrom
fix-wayfind-status-slug-flag

Conversation

@azalio

@azalio azalio commented Aug 10, 2026

Copy link
Copy Markdown
Owner

The /map-wayfind skill body told the operator to run wayfind_status <slug> in two places, but the CLI declares that argument as --slug. Following the skill verbatim fails:

wayfind_runner.py: error: unrecognized arguments: review-verdict-ledger

Hit while actually charting a map — both affected lines are on the happy path: the chart step that shows the map after creation, and the handoff-eligibility check in handoff mode.

wayfind-reference.md already documented wayfind_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.md and wayfind-reference.md against its argparse definition in wayfind_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_handoff all match their parsers. wayfind_status was the only mismatch.

Verification

Edited templates_src/**/*.jinja and re-rendered; make check-render reports the generated trees match.

Summary by CodeRabbit

  • Documentation
    • Updated wayfind_status command examples to use the explicit --slug <slug> option.
    • Aligned guidance across generated and source documentation for chart stopping and handoff eligibility checks.

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.
@coderabbitai

coderabbitai Bot commented Aug 10, 2026

Copy link
Copy Markdown

Review Change Stack

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro Plus

Run ID: 1c854202-0333-4ca2-9cc1-3d5e87e21a7f

📥 Commits

Reviewing files that changed from the base of the PR and between b3b1d93 and a8fb3ec.

📒 Files selected for processing (3)
  • .claude/skills/map-wayfind/SKILL.md
  • src/mapify_cli/templates/skills/map-wayfind/SKILL.md
  • src/mapify_cli/templates_src/skills/map-wayfind/SKILL.md.jinja

📝 Walkthrough

Walkthrough

The PR updates two wayfind_status command examples in each of three Wayfind skill documentation sources. Each example changes from positional slug syntax to the explicit --slug <slug> option.

Changes

Wayfind status syntax

Layer / File(s) Summary
Update status command examples
.claude/skills/map-wayfind/SKILL.md, src/mapify_cli/templates/skills/map-wayfind/SKILL.md, src/mapify_cli/templates_src/skills/map-wayfind/SKILL.md.jinja
The chart stop step and handoff eligibility check use wayfind_status --slug <slug> instead of positional slug syntax.

Estimated code review effort: 1 (Trivial) | ~2 minutes

Possibly related PRs

Poem

A rabbit checks the slug with care,
An option flag now leads it there.
Two status calls hop into line,
Across each template, neat and fine.
--slug makes the path clear!

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title clearly and concisely describes the documented change from positional slug syntax to the --slug option.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch fix-wayfind-status-slug-flag

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.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

@azalio
azalio merged commit 2a76fbc into main Aug 10, 2026
7 checks passed
azalio pushed a commit that referenced this pull request Aug 12, 2026
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).
azalio added a commit that referenced this pull request Aug 12, 2026
…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>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant