Skip to content

docs: clarify the standards library starting points - #5

Merged
MrScripty merged 3 commits into
mainfrom
docs/readme-clarity-20261015
Oct 22, 2026
Merged

MrScripty merged 3 commits into
mainfrom
docs/readme-clarity-20261015

Conversation

@MrScripty

@MrScripty MrScripty commented Oct 15, 2026 •

Copy link
Copy Markdown
Owner

Summary

  • Clarify the library's purpose and the reading path through Core Standards and the Router.
  • Preserve Router authority over applicability/dependencies and explain verification limits.
  • Link the existing Engine connection guide and refresh the generated verification-input companion.

Exact reviewed candidate

  • Head: 4d6e755a886061e18582120e54e7f7e9bfe6fe26.
  • Base: 0fc92a24279f3a4574c80af0a2f31ea347a21597, the normal merge of product PR #4.
  • Exactly two changed files: README.md and evaluation/standards-effectiveness/generated/suite-inputs.json; no source, tests, validators, thresholds, or workflow changes.
  • The branch retains its preceding head and current main as parents. No force push or branch deletion.
  • Marked ready for review during the 2026-10-21 PR disposition pass. Reapplying its intended base main reconciled previously stale PR head/base metadata and queued exact-head CI.

Current validation

  • Independent GPT-6.1 Sol High review of the full exact-head diff: no findings requiring changes. README preserves authority/diagnostic meaning; all 13 relative links resolve to tracked regular files and describe their targets accurately.
  • Manifest retains all 469 previous file entries and uses; only the connection-guide link-target entry and README digest change. All 470 present-file digests match; canonical generation matches the committed manifest byte-for-byte.
  • Isolated Python 3.12.14 environment installed from the unchanged lock with --require-hashes --only-binary=:all:.
  • Engine public verify_repository, refresh_verification_inputs: false: 73 suites / 121 checks passed.
  • Contract projection --check and git diff --check: passed.
  • test_proposal_analysis_uses_one_projection_and_one_verified_base: passed (1 test, 8.073 seconds) in a separate clone with candidate main.
  • A first focused invocation in a detached worktree failed class setup because no local accepted main existed; a broad local run from that worktree was interrupted. Those logs are retained; no full local Engine-suite pass is claimed.

Hosted CI and retained failure evidence

  • Exact-head run 39676099972 passed for reviewed head 4d6e755a886061e18582120e54e7f7e9bfe6fe26 against 0fc92a24279f3a4574c80af0a2f31ea347a21597: all workflow unit suites, including 636 Engine tests, and complete registered verification (73/73 suites).
  • Post-merge push run 39709176048 completed successfully on normal merge 73bd032b8840656e104c3748c4b73cc197bffb17: all workflow unit suites, including 636 Engine tests, and complete registered verification (73/73 suites; zero failures or blocked suites).
  • Original old-head failure: 613 Engine tests, one AUTHORING.INVALID_SEMANTIC_REVISION fixture assertion; complete repository verification was not reached.
  • Old main-base failure exhibited that same fixture failure.
  • The fixture repair was reviewed and retained in PR Implement opt-in persistent MCP interaction observability #4; its historical green run is supporting history, not a substitute for this exact-head run.
  • The automated draft-skipped review is not counted as source approval.

README SHA-256: 52aea11468d154104aa294717598f550dd1170e7ce144d5ff38a1752177a01a6.
Generated companion SHA-256: e6cdf2bfa6a4eb0e9650a2ae6401c929be3e33240a9cb8de98c41087261c3039.

Final disposition

Merged normally as 73bd032b8840656e104c3748c4b73cc197bffb17, with parents 0fc92a24279f3a4574c80af0a2f31ea347a21597 and 4d6e755a886061e18582120e54e7f7e9bfe6fe26. The merge tree is the exact independently reviewed candidate tree b9c9e7b3ff0ab992cb91d4cb02da514a5dc87386. Main and PR merged state were read back and verified; the source branch remains at its reviewed head. No open PRs remain. No branches deleted, history rewritten, or unrelated local work changed. No unresolved blockers.

@coderabbitai

coderabbitai Bot commented Oct 15, 2026 •

Copy link
Copy Markdown

Important

  • 🔍 Trigger review

This repository does not receive automatic reviews because it has fewer than 10 stars.

⚙️ Run configuration
  • Configuration used: defaults
  • Review profile: CHILL
  • Plan: Advanced
  • Run ID: f42cf22a-aac5-4417-bd22-b68fc7bda6c1
  • Autofix · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

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.

Regenerate README verification inputs through the supported Engine after integrating the reviewed observability and snapshot-revision fixture repair. Keep the README draft separate.
@MrScripty
MrScripty marked this pull request as ready for review October 21, 2026 21:05

Copy link
Copy Markdown
Owner Author

PR disposition checkpoint (2026-10-21): preserve open pending hosted validation.

Reviewed exact head 4d6e755a886061e18582120e54e7f7e9bfe6fe26 against 0fc92a24279f3a4574c80af0a2f31ea347a21597 separately from merged product PR #4. Independent GPT-6.1 Sol High review found no required changes; all 13 README links resolve, and canonical manifest generation matches committed bytes with no input loss or weakened gates. Local locked-Python verification passed 73 suites / 121 checks, contract freshness, whitespace checks, and the previously failing focused regression (1 test, 8.073s).

The stale PR head/base metadata was reconciled by reapplying its intended base main; the PR is ready for review. Exact-head CI run 39676099972 is currently queued with no runner assigned and no steps started. This is pending evidence, not a CI pass or failure. Merge only after this run succeeds and refs remain the reviewed revisions. No merge or closure attempted, and no branches deleted or history rewritten.

The old 613-test failure remains retained. Draft-skipped automated review is not counted as approval.

@MrScripty
MrScripty merged commit 73bd032 into main Oct 22, 2026
2 checks passed

Copy link
Copy Markdown
Owner Author

Disposition complete: merged by normal merge as 73bd032b8840656e104c3748c4b73cc197bffb17.

The head and target remained exactly the independently reviewed revisions; no review threads or additional required gates were outstanding. Exact-head CI passed before merge. Post-merge CI then completed successfully on the merge commit: all workflow unit suites, including 636 Engine tests, and complete verification with 73/73 suites passed, zero failures, zero blocked.

Verified both merge parents and exact tree equality with the reviewed candidate. Main points to the merge, this PR is closed as merged, the source branch is preserved, and no open PRs remain. Historical failure evidence and unrelated local work were preserved. No unresolved blockers.

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