Skip to content

docs(spec): stop quoting a corpus vector count - #1024

Merged
jdx merged 1 commit into
cursor/default-if-66dbfrom
cursor/drop-vector-count-66db
Aug 18, 2026
Merged

docs(spec): stop quoting a corpus vector count#1024
jdx merged 1 commit into
cursor/default-if-66dbfrom
cursor/drop-vector-count-66db

Conversation

@jdx

@jdx jdx commented Aug 18, 2026

Copy link
Copy Markdown
Owner

The argv grammar page quoted how many corpus vectors usage-lib agrees with. That count goes stale on every new vector file.

Agreement is already measured on each run (conformance/tests/reference.rs) rather than asserted in prose, so the page now says "every vector" instead of a number.

This comment was generated by Claude Code.

Open in Web Open in Cursor 

@coderabbitai

coderabbitai Bot commented Aug 18, 2026

Copy link
Copy Markdown

Important

Review skipped

Auto reviews are disabled on base/target branches other than the default branch.

Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

⚙️ Run configuration

Configuration used: Central YAML (base), Organization UI (inherited)

Review profile: CHILL

Plan: Pro Plus

Run ID: 6df867c3-414e-493f-9365-414611d38680

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

  • 🔍 Trigger review

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.

@cursor
cursor Bot marked this pull request as ready for review August 18, 2026 13:26
@github-actions

github-actions Bot commented Aug 18, 2026

Copy link
Copy Markdown
Contributor

Instruction counts

benchmark trend instructions Δ wall (min) Δ
markdown ▁▁▂▂▄▂▁▂▂█ 196,141,583 → 196,799,812 +0.34% 18.22 → 17.30ms -5.02%
startup █▄███████▁ 829,332 → 825,005 -0.52% 0.84 → 0.86ms +2.01%

No instruction-count regression above 1%.

Only instruction counts gate. Wall clock is shown for context — on identical hardware it moves 4-20% run to run.

Measured by tak — instruction-counted CLI benchmarks, stored in this repository's git notes.

Shadow comparison

Parsing mise use -g node@20 against a shadow of mise's committed spec.
Reported, not gated: the shadow grows as the derive learns to express more, so
what to watch is the ratio rather than either column.

framework instructions, cold parse vs usage
usage 4220
argh 6292 1.5x
clap 5895248 1396x
bpaf 21917796 5193x
                                              min       p01       p10    median
usage-rs: argv -> struct                      192       195       198       202  ns
argh: argv -> struct                          285       289       294       303  ns
clap: build tree + parse -> struct         485778    487396    490726    496982  ns
bpaf: build parser + parse -> struct      1604914   1604914   1612504   1631420  ns

usage: argv -> struct                             202 ns      0.20 µs
clap: build tree + parse -> struct             496881 ns    496.88 µs
clap: parse -> struct, tree reused              23481 ns     23.48 µs
clap: build tree only                          308240 ns    308.24 µs

d4b0747c7287 vs df69276f650a · measured on the runner, not pushed to the history.

@cursor
cursor Bot force-pushed the cursor/drop-vector-count-66db branch 2 times, most recently from 3c8c471 to 862d214 Compare August 18, 2026 13:48
The number goes stale every time a vector is added. Agreement is already
measured on each run rather than asserted in the grammar page.

Co-authored-by: jeff <jeff@jdx.dev>
@cursor
cursor Bot force-pushed the cursor/drop-vector-count-66db branch from 862d214 to d4b0747 Compare August 18, 2026 14:01
@jdx
jdx merged commit 49f9e4a into main Aug 18, 2026
10 checks passed
@jdx
jdx deleted the cursor/drop-vector-count-66db branch August 18, 2026 15:05
jdx added a commit that referenced this pull request Aug 19, 2026
<!-- CURSOR_AGENT_PR_BODY_BEGIN -->
`external_subcommand` (#1021) and `default_if` (#1023) are in the
parser, the derive, and the corpus. PLAN.md still listed them as open
clap gaps.

This ticks those items, drops them from the fleet table, and splits
`multicall` from `no_binary_name` so the next PR can close one without
pretending the other landed. `no_binary_name` stays out of scope until a
fleet CLI needs it.

Also stops quoting a corpus vector count in PLAN.md and `go/README.md`,
matching the argv grammar page (#1024): agreement is measured on each
run rather than asserted as "154".

Stacked next: #1028 (`multicall`).

_This comment was generated by Claude Code._
<!-- CURSOR_AGENT_PR_BODY_END -->

<div><a
href="https://cursor.com/agents/bc-4de7e59e-aa2e-4ad0-a1d0-1c2e69c166db?cursor_ref=pr_footer&cursor_cta=open_in_web"><picture><source
media="(prefers-color-scheme: dark)"
srcset="https://cursor.com/assets/images/open-in-web-dark.png"><source
media="(prefers-color-scheme: light)"
srcset="https://cursor.com/assets/images/open-in-web-light.png"><img
alt="Open in Web" width="114" height="28"
src="https://cursor.com/assets/images/open-in-web-dark.png"></picture></a>&nbsp;<a
href="https://cursor.com/background-agent?bcId=bc-4de7e59e-aa2e-4ad0-a1d0-1c2e69c166db&cursor_ref=pr_footer&cursor_cta=open_in_cursor"><picture><source
media="(prefers-color-scheme: dark)"
srcset="https://cursor.com/assets/images/open-in-cursor-dark.png"><source
media="(prefers-color-scheme: light)"
srcset="https://cursor.com/assets/images/open-in-cursor-light.png"><img
alt="Open in Cursor" width="131" height="28"
src="https://cursor.com/assets/images/open-in-cursor-dark.png"></picture></a>&nbsp;</div>



<!-- This is an auto-generated comment: release notes by coderabbit.ai
-->
## Summary by CodeRabbit

* **New Features**
* Added BusyBox-style multicall support, selecting subcommands from the
executable name.
* Added support for paths, Windows `.exe` suffixes, dispatcher names,
symlink-style invocations, and external subcommands.
* Added aliases, hidden choices, and optional case-insensitive value
matching.
* Added multicall configuration across supported integrations and
generated specifications.

* **Bug Fixes**
* Added validation for invalid multicall configurations and clearer
unknown-applet behavior.

* **Documentation**
* Documented multicall behavior, value choices, supported invocation
patterns, and conformance status.
<!-- end of auto-generated comment: release notes by coderabbit.ai -->

---------

Co-authored-by: Cursor Agent <cursoragent@cursor.com>
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.

2 participants