Skip to content

chore(quality): align the extension docs and cover the push routes - #24

Merged
T-Crypt merged 3 commits into
oh-my-pocketfrom
quality/2026-10-16
Oct 17, 2026
Merged

T-Crypt merged 3 commits into
oh-my-pocketfrom
quality/2026-10-16

Conversation

@T-Crypt

@T-Crypt T-Crypt commented Oct 17, 2026

Copy link
Copy Markdown
Owner

Why

A quality pass over oh-my-pocket found two documents that no longer matched the code, and one user-facing route group with no test. The extension docs drifted as modules were added, and the push routes were untested.

What changed

  • docs/map.md lists every built-in extension in install order, including routers, read-economy, loop-guard, pruner, tool-repair, and today.
  • docs/extensions.md names the host members a drop-in can use, including envFor, routerFor, loopGuard, pip, pipCompaction, and pruner.
  • test/push-routes.test.ts covers the /api/push routes, the largest untested gap among core server modules.
  • docs/triage-2026-10-16.md records where the 20 open upstream issues stand for this fork.

Scope

This covers docs accuracy, one new test file, and a triage note. It leaves the code alone and adds no dependencies. The README's battery test counts and its extensions summary row are left as they are.

Blast radius

Docs and one new test file. Nothing under src/ or web/ changes, so no running server is affected.

Verification

  • npm test passes 643 of 643 tests, including the 3 new ones.
  • npm run check, npm run lint, and npm run check:web pass.
  • node --test test/push-routes.test.ts passes 3 of 3.
  • npx prettier --check passes on every changed file. The full npm run format:check warns on an untracked opencode.json that this worktree holds and the repository does not.

@T-Crypt
T-Crypt marked this pull request as ready for review October 17, 2026 01:32
@T-Crypt
T-Crypt merged commit a754d06 into oh-my-pocket Oct 17, 2026
2 checks passed
@T-Crypt
T-Crypt deleted the quality/2026-10-16 branch October 17, 2026 02:06
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