Skip to content

docs(cursor): fix broken intra-doc links breaking release-check#107

Merged
akesling merged 1 commit into
mainfrom
fix/cursor-broken-doc-links
Jun 22, 2026
Merged

docs(cursor): fix broken intra-doc links breaking release-check#107
akesling merged 1 commit into
mainfrom
fix/cursor-broken-doc-links

Conversation

@akesling

@akesling akesling commented Jun 22, 2026

Copy link
Copy Markdown
Contributor

Problem

just release-check (→ scripts/release.sh, which runs RUSTDOCFLAGS="-D warnings" cargo doc --workspace --no-deps) fails on main:

error: unresolved link to `list_composers`        crates/toolpath-cursor/src/io.rs:55
error: unresolved link to `composer_has_bubbles`   crates/toolpath-cursor/src/reader.rs:75
error: could not document `toolpath-cursor`

Cause

Both links target methods, but a bare [`name`] only resolves free items in rustdoc's scope — methods need a type qualifier. Under -D rustdoc::broken-intra-doc-links this is a hard error. The methods exist (CursorIO::list_composers at io.rs:64, DbReader::composer_has_bubbles at reader.rs:181); the links just weren't qualified.

Fix

Qualify both with Self::, matching the crate's existing [CursorIO::list_composers] style.

Verification

  • RUSTDOCFLAGS="-D warnings" cargo doc --workspace --no-deps → clean (the exact release-check doc gate)
  • cargo test -p toolpath-cursor --doc → passes

Docs-only change; no code, no version bump.


View with Codesmith Autofix with Codesmith
Need help on this PR? Tag /codesmith with what you need. Autofix is disabled.

`[`list_composers`]` (io.rs) and `[`composer_has_bubbles`]` (reader.rs)
are methods, so a bare link can't resolve in rustdoc's scope and fails
under `-D rustdoc::broken-intra-doc-links`. Qualify both with `Self::`,
matching the existing `[`CursorIO::list_composers`]` style elsewhere in
the crate. Fixes `cargo doc --workspace` (the release-check doc gate).
@github-actions

Copy link
Copy Markdown

🔍 Preview deployed: https://937de225.toolpath.pages.dev

@akesling akesling merged commit 00f358e into main Jun 22, 2026
2 checks passed
@akesling akesling deleted the fix/cursor-broken-doc-links branch June 22, 2026 19:59
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