docs: sweep stale ADR-0015/0016 prose; unify CI test install path#146
Merged
Conversation
PR-D from the 2026-05-28 tech-debt audit. Documentation + CI hygiene following the two rip-and-replaces; no runtime code change. CI (ci.yml) — the `test` matrix special-cased Node 22 with a native tree-sitter grammar build (node-gyp install + scripts-enabled postinstall + OCH_NATIVE_PARSER=1), while Node 24 already ran --ignore-scripts and passed the full suite. Since parsing is WASM-only on every Node version (ADR 0015) and the remaining native deps (@duckdb/node-api, @ladybugdb/core, onnxruntime-node) ship prebuilds, both lanes now share one `pnpm install --frozen-lockfile --ignore-scripts` step. Drops the node-gyp install, the dual install steps, and the dead OCH_NATIVE_PARSER env injection. mise.toml — node-gyp comment corrected (it's a prebuild fallback for @duckdb/node-api / onnxruntime-node, not a tree-sitter build dep). Docs (17 files) — swept two stale narratives: - ADR 0015 (WASM-only parsing): removed "Node 22 native opt-in", "optional native tree-sitter addon", and native-build instructions for parsing across install.mdx, overview.md, install.md, cli.md, troubleshooting.md. Kept genuine @duckdb/node-api / onnxruntime-node native-build troubleshooting. - ADR 0016 (lbug-only graph): removed the CODEHUB_STORE env var, the backend probe, graph.duckdb, and the dual-artifact mtime arbitration from storage-backend.md, configuration.md, error-codes.md, determinism.md, monorepo-map.md, indexing-a-repo.md, quick-start.md, what-is-opencodehub.md, README.md, AGENTS.md, packages/storage/README.md, packages/cli/README.md. The segregated IGraphStore/ITemporalStore community-adapter escape hatch is preserved (it's the deliberate v1.0 contract). --skip-native help/docs now read "(duckdb / lbug)". - AGENTS.md storage section had drifted from its "synchronized with CLAUDE.md" header — brought back into parity (CLAUDE.md was already correct and is untouched). - overview.md ADR-reference table: marked 0013 superseded, added 0015 and 0016 rows. Validation: banned-strings PASS, ci.yml valid YAML, mise.toml valid TOML. Residual CODEHUB_STORE/graph.duckdb grep hits are all "removed in ADR 0016" prose (intentional).
Merged
theagenticguy
pushed a commit
that referenced
this pull request
May 28, 2026
🤖 Automated release via release-please --- <details><summary>analysis: 0.3.1</summary> ## [0.3.1](analysis-v0.3.0...analysis-v0.3.1) (2026-05-28) ### Dependencies * The following workspace dependencies were updated * dependencies * @opencodehub/storage bumped to 0.2.1 * @opencodehub/wiki bumped to 0.2.1 </details> <details><summary>cli: 0.5.2</summary> ## [0.5.2](cli-v0.5.1...cli-v0.5.2) (2026-05-28) ### Bug Fixes * harden SCIP proto-reader bounds; drop dead native tree-sitter doctor probe ([#138](#138)) ([b1a4772](b1a4772)) ### Performance * **ingestion:** O(N) complexity lookup; fix sql hint; reuse openStoreForCommand ([#142](#142)) ([976b877](976b877)) ### Documentation * sweep stale ADR-0015/0016 prose; unify CI test install path ([#146](#146)) ([3b2e05e](3b2e05e)) ### Refactoring * drop dead materialize() + cross-backend parity script (−425 LOC) ([#141](#141)) ([216121a](216121a)) ### Dependencies * The following workspace dependencies were updated * dependencies * @opencodehub/analysis bumped to 0.3.1 * @opencodehub/ingestion bumped to 0.4.2 * @opencodehub/mcp bumped to 0.4.1 * @opencodehub/pack bumped to 0.2.1 * @opencodehub/search bumped to 0.2.1 * @opencodehub/storage bumped to 0.2.1 * @opencodehub/wiki bumped to 0.2.1 </details> <details><summary>cobol-proleap: 0.1.6</summary> ## [0.1.6](cobol-proleap-v0.1.5...cobol-proleap-v0.1.6) (2026-05-28) ### Dependencies * The following workspace dependencies were updated * dependencies * @opencodehub/ingestion bumped to 0.4.2 </details> <details><summary>ingestion: 0.4.2</summary> ## [0.4.2](ingestion-v0.4.1...ingestion-v0.4.2) (2026-05-28) ### Performance * **ingestion:** O(N) complexity lookup; fix sql hint; reuse openStoreForCommand ([#142](#142)) ([976b877](976b877)) ### Refactoring * **ingestion:** collapse 3 IndexerKind switches into LANG_REGISTRY ([#143](#143)) ([dea4001](dea4001)) ### Dependencies * The following workspace dependencies were updated * dependencies * @opencodehub/analysis bumped to 0.3.1 * @opencodehub/scip-ingest bumped to 0.2.3 * @opencodehub/storage bumped to 0.2.1 </details> <details><summary>mcp: 0.4.1</summary> ## [0.4.1](mcp-v0.4.0...mcp-v0.4.1) (2026-05-28) ### Performance * **ingestion:** O(N) complexity lookup; fix sql hint; reuse openStoreForCommand ([#142](#142)) ([976b877](976b877)) ### Dependencies * The following workspace dependencies were updated * dependencies * @opencodehub/analysis bumped to 0.3.1 * @opencodehub/pack bumped to 0.2.1 * @opencodehub/search bumped to 0.2.1 * @opencodehub/storage bumped to 0.2.1 </details> <details><summary>pack: 0.2.1</summary> ## [0.2.1](pack-v0.2.0...pack-v0.2.1) (2026-05-28) ### Dependencies * The following workspace dependencies were updated * dependencies * @opencodehub/analysis bumped to 0.3.1 * @opencodehub/ingestion bumped to 0.4.2 * @opencodehub/storage bumped to 0.2.1 </details> <details><summary>scip-ingest: 0.2.3</summary> ## [0.2.3](scip-ingest-v0.2.2...scip-ingest-v0.2.3) (2026-05-28) ### Bug Fixes * harden SCIP proto-reader bounds; drop dead native tree-sitter doctor probe ([#138](#138)) ([b1a4772](b1a4772)) ### Refactoring * drop dead materialize() + cross-backend parity script (−425 LOC) ([#141](#141)) ([216121a](216121a)) ### Dependencies * The following workspace dependencies were updated * dependencies * @opencodehub/analysis bumped to 0.3.1 </details> <details><summary>search: 0.2.1</summary> ## [0.2.1](search-v0.2.0...search-v0.2.1) (2026-05-28) ### Dependencies * The following workspace dependencies were updated * dependencies * @opencodehub/storage bumped to 0.2.1 </details> <details><summary>storage: 0.2.1</summary> ## [0.2.1](storage-v0.2.0...storage-v0.2.1) (2026-05-28) ### Documentation * sweep stale ADR-0015/0016 prose; unify CI test install path ([#146](#146)) ([3b2e05e](3b2e05e)) ### Refactoring * drop dead materialize() + cross-backend parity script (−425 LOC) ([#141](#141)) ([216121a](216121a)) </details> <details><summary>wiki: 0.2.1</summary> ## [0.2.1](wiki-v0.2.0...wiki-v0.2.1) (2026-05-28) ### Dependencies * The following workspace dependencies were updated * dependencies * @opencodehub/storage bumped to 0.2.1 </details> <details><summary>root: 0.6.2</summary> ## [0.6.2](root-v0.6.1...root-v0.6.2) (2026-05-28) ### Bug Fixes * harden SCIP proto-reader bounds; drop dead native tree-sitter doctor probe ([#138](#138)) ([b1a4772](b1a4772)) ### Performance * **ingestion:** O(N) complexity lookup; fix sql hint; reuse openStoreForCommand ([#142](#142)) ([976b877](976b877)) ### Documentation * **repo:** add 2 ERPAVal durable lessons from PR [#138](#138) Compound phase ([#140](#140)) ([ffd2435](ffd2435)) * **repo:** add collapse-parallel-switches-into-record-registry lesson ([#144](#144)) ([b1685f5](b1685f5)) * sweep stale ADR-0015/0016 prose; unify CI test install path ([#146](#146)) ([3b2e05e](3b2e05e)) ### Refactoring * drop dead materialize() + cross-backend parity script (−425 LOC) ([#141](#141)) ([216121a](216121a)) * **ingestion:** collapse 3 IndexerKind switches into LANG_REGISTRY ([#143](#143)) ([dea4001](dea4001)) </details> --- This PR was generated with [Release Please](https://github.com/googleapis/release-please). See [documentation](https://github.com/googleapis/release-please#release-please). Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
PR-D from the 2026-05-28 tech-debt audit (
.erpaval/sessions/session-88b46e/verdict-memo.md). Documentation + CI hygiene following the two rip-and-replaces (ADR 0015 WASM-only parser, ADR 0016 lbug-only graph). No runtime code change.CI (
ci.yml) — unify the test install pathThe
testmatrix special-cased Node 22 with a native tree-sitter grammar build (node-gypinstall + scripts-enabled postinstall +OCH_NATIVE_PARSER=1), while Node 24 already ran--ignore-scriptsand passed the full suite. Since parsing is WASM-only on every Node version and the remaining native deps (@duckdb/node-api,@ladybugdb/core,onnxruntime-node) ship prebuilds, both lanes now share onepnpm install --frozen-lockfile --ignore-scriptsstep. Drops thenode-gypinstall, the dual install steps, and the deadOCH_NATIVE_PARSERenv injection.mise.toml—node-gypcomment corrected (prebuild fallback for DuckDB/onnx, not a tree-sitter build dep).Docs (17 files) — two stale narratives swept
install.mdx,overview.md,install.md,cli.md,troubleshooting.md. Kept genuine DuckDB/onnx native-build troubleshooting.CODEHUB_STORE, the backend probe,graph.duckdb, and the dual-artifact mtime arbitration from 12 files. The segregatedIGraphStore/ITemporalStorecommunity-adapter escape hatch is preserved (deliberate v1.0 contract).--skip-nativehelp/docs now read(duckdb / lbug).Test plan
banned-strings— PASSci.ymlvalid YAML,mise.tomlvalid TOMLCODEHUB_STORE/graph.duckdbgrep hits are all "removed in ADR 0016" prose (intentional)--no-verify) —verdict/typecheck/testall green--ignore-scripts)First PR pushed without
--no-verifysince the verdict-gate fix (#145) landed.