Skip to content

docs: clarify frame state ownership and refresh knowledge - #378

Merged
WH-2099 merged 1 commit into
mainfrom
docs/knowledge-review-20261029
Oct 29, 2026
Merged

WH-2099 merged 1 commit into
mainfrom
docs/knowledge-review-20261029

Conversation

@laipz8200

@laipz8200 laipz8200 commented Oct 29, 2026 •

Copy link
Copy Markdown
Member

Important

  1. Make sure you have read our contribution guidelines
  2. Search existing issues and pull requests to confirm this change is not a duplicate
  3. Open or identify the issue this pull request resolves or advances
  4. Use a Conventional Commits title for this pull request, and mark breaking changes with !
  5. Remember that the pull request title will become the squash commit message
  6. If CLA Assistant prompts you, sign CLA.md in the pull request conversation

Related Issue

Closes #377

Summary

Review all 12 active knowledge pages against fetched default-branch revision a315092e0a84160aaf336091e6e547445208324e in a fresh worktree, following docs/README.md, docs/maintenance.md, and CONTRIBUTING.md. Refresh each page's review timestamp after checking its implementation, callers, tests, tooling, navigation, and relevant external references.

Correct the README's state-ownership summary and shorten detailed frame-failure notes into lifecycle guidance with links to implementation and tests. No behavior, test, fixture, archive, or legal-content changes.

Verify that issue #199 and PRs #200, #266, and #271 remain open; active debt remains unresolved. The prior inspection (#351 / #352) is merged.

Validation

  • Documentation validation: 29 passed, including maintained scope, review metadata, freshness, local links, anchors and navigation.
  • just test: 1,063 passed on Python 3.12.16, with two existing pandas deprecation warnings.
  • just check, uv run prek run --all-files, and git diff --check: passed.
  • Timestamp automation tests: 20 passed; PR checks: 21 passed; example configuration tests: 6 passed. Both Slim example --help commands passed.
  • Focused frame observation/failure/layer and model dispatch/accumulator checks: 80 passed.
  • A fresh context-free reviewer independently checked all 12 pages, preserved the precise Exception qualification in failure ordering, and passed 33 documentation/frame-failure tests.
  • A separate fresh final naming pass completed without renames.

GitHub CI passed, including Python 3.12/3.13 tests, Check, PR title validation, document timestamp automation, CodeQL and CLA. No new CLA signature action was required.

Python 3.13, performance benchmarks, and live model/daemon/Redis integrations were not run locally.

Checklist

  • This pull request links the issue it resolves or advances
  • This pull request title follows Conventional Commits, and any breaking change is marked with !
  • If CLA Assistant prompted me, I signed CLA.md in the pull request conversation

No new CLA signature prompt has been received; the conditional checklist item does not apply locally.

@laipz8200 laipz8200 added the documentation Improvements or additions to documentation label Oct 29, 2026
@WH-2099
WH-2099 merged commit 9f85928 into main Oct 29, 2026
13 checks passed
@WH-2099
WH-2099 deleted the docs/knowledge-review-20261029 branch October 29, 2026 06:44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

documentation Improvements or additions to documentation

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Review active repository knowledge for October 29

2 participants