Skip to content

perf: index law dispatch by canonical structure - #196

Merged
akcorca merged 1 commit into
mainfrom
codex/issue-190-indexed-law-dispatch
Aug 10, 2026
Merged

perf: index law dispatch by canonical structure#196
akcorca merged 1 commit into
mainfrom
codex/issue-190-indexed-law-dispatch

Conversation

@akcorca

@akcorca akcorca commented Aug 10, 2026

Copy link
Copy Markdown
Collaborator

Closes #190

Summary

  • compile immutable root/mandatory-structure keys for every canonical law form
  • dispatch only structurally compatible candidates while retaining an explicit fallback for undiscriminated variadic forms
  • keep exhaustive unification as a test oracle and prove candidate completeness across all compiled forms
  • add pure 100- and 500-pack scale tests plus a 20-rule-per-document budget
  • remove the production exhaustive scan and its legacy-audit finding

Results

  • the 61-document fixture drops rule visits from 2,240 to 950 (57.6%)
  • cargo clippy --workspace --all-targets -- -D warnings
  • cargo test --workspace (125 tests)
  • bun run typecheck and bun test
  • bun run legacy:audit (only v0.19 4/6: Centralize meaning decisions and resolution plans #191 remains)
  • awiki lint
  • 501-document local timing remained well below latency limits; Apple Silicon RSS is diagnostic per policy and the Linux CI gate is authoritative

Release WASM was rebuilt on the designated x86_64 Linux builder.

@coderabbitai

coderabbitai Bot commented Aug 10, 2026

Copy link
Copy Markdown

Important

Review skipped

Auto reviews are disabled on this repository. 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: Organization UI

Review profile: CHILL

Plan: Pro Plus

Run ID: 060a1f7a-7255-473a-828d-2f6683b452da

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.

@akcorca
akcorca merged commit e5c97c8 into main Aug 10, 2026
2 checks passed
@akcorca
akcorca deleted the codex/issue-190-indexed-law-dispatch branch August 10, 2026 10:58
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.

v0.19 3/6: Index law dispatch for hundreds of packs

1 participant