8000 docs: fix Rust Quick Start rule and entity-ID examples by adityamukho · Pull Request #893 · project-minigraf/minigraf · GitHub
[go: up one dir, main page]

Skip to content

docs: fix Rust Quick Start rule and entity-ID examples - #893

Open
adityamukho wants to merge 1 commit into
mainfrom
fix/884-rust-quick-start
Open

adityamukho wants to merge 1 commit into
mainfrom
fix/884-rust-quick-start

Conversation

@adityamukho
@adityamukho adityamukho commented Nov 13, 2026 •
Copy link
Copy Markdown
Collaborator

The README passes two rule forms to one execute() call, which returns PRS-077, and uses undefined alice_id/bob_id variables. Register each rule clause separately, demonstrate the reachability query, and define keyword IDs using Minigraf's public EntityId alias. Also define the ID in the related API-010 example and record the correction in the changelog. No runtime code, dependencies, or test inventory changes.

Validation:

  • Reproduced the undefined-variable compiler errors, then PRS-077 after supplying IDs.
  • Compiled and ran the corrected README in a Result-returning function wrapper with assertions for Bob's reachability and Alice/Bob prepared-query results. The corrected API-010 and docs-site examples also compile and run against main.
  • cargo test -j 2, cargo clippy -j 2 -- -D warnings, cargo fmt --check, and git diff --check passed.

Audited crate docs, the retired wiki, and minigraf-docs for duplicate broken examples. The wiki contains redirects. The docs-site correction is project-minigraf/minigraf-docs#98; the equivalent v3 fix is #892.

Fixes #884. Claude Opus independently reviewed this patch and found it correct. Its findings in the related docs-site PR were fixed and received a clean follow-up review before pushing those corrections.

@codecov
codecov Bot commented Nov 13, 2026
Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.

📢 Thoughts on this report? Let us know!

This branch has not been deployed

No deployments
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.

README Rust Quick Start: recursive rule returns PRS-077, and alice_id/bob_id are undefined

1 participant

0