From fff441196cebc63e61e7dc96f7c623324d198660 Mon Sep 17 00:00:00 2001 From: Andrew Altshuler Date: Sun, 21 Jun 2026 00:11:48 +0300 Subject: [PATCH] =?UTF-8?q?docs(dev):=20update=20coherence=20ledger=20?= =?UTF-8?q?=E2=80=94=20cookbooks=20drift=20resolved,=20omnigraph-ts=20mech?= =?UTF-8?q?anism=20(#294)?= MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit - omnigraph-cookbooks `bearer_token_env` chain: RESOLVED by cookbooks PR #26 (deleted docs/best-practices.md in the 0.7 restructure). - omnigraph-ts catalog `mcp.expose` description: documented why there is no hand-fix — the SDK syncs openapi.json from a *tagged* omnigraph release, and the fix landed on main after the v0.7.1 tag, so it flows in on the next SDK version bump (v0.7.2+) rather than an out-of-band patch. Claude-Session: https://claude.ai/code/session_01FQ1Hf4eXLsJmeLUkTYBEw7 Co-authored-by: Claude Opus 4.8 (1M context) --- docs/dev/docs-issues.md | 24 +++++++++++++----------- 1 file changed, 13 insertions(+), 11 deletions(-) diff --git a/docs/dev/docs-issues.md b/docs/dev/docs-issues.md index d045f4b..c0a4fdb 100644 --- a/docs/dev/docs-issues.md +++ b/docs/dev/docs-issues.md @@ -40,17 +40,19 @@ fixed. Branch `docs/user-coherence-0-7-1`. `apply_schema_update_and_dependent_query_in_one_run`, `apply_blocks_graph_delete_without_approval`). Update these comments in a cluster-crate change. -- **Cross-repo drift from this sweep** (separate repos — track here, fix in a - follow-up in each repo): - - `omnigraph-ts` SDK ships a stale generated `spec/openapi.json` + - `packages/sdk/src/generated/types.gen.ts` still describing the `GET /queries` - catalog as the `mcp.expose` subset. Regenerate from this repo's - `openapi.json` once the SDK's deferred refresh happens (the SDK is known to - lag the API by design). - - `omnigraph-cookbooks/docs/best-practices.md` (~line 372) still describes - client-side auth as resolving through the removed `bearer_token_env` chain. - Update to the keyed-credential model (`OMNIGRAPH_TOKEN_` → - credentials file → `OMNIGRAPH_BEARER_TOKEN`). +- **Cross-repo drift from this sweep** (separate repos): + - `omnigraph-ts` SDK — its generated `spec/openapi.json` + + `packages/sdk/src/generated/types.gen.ts` still describe the `GET /queries` + catalog as the `mcp.expose` subset. **No hand-fix:** the SDK's + `scripts/sync-spec.ts` pulls openapi.json from a *tagged* omnigraph release + (`/omnigraph/v{version}/openapi.json`), and the catalog fix landed on main + *after* the v0.7.1 tag — so it is in no tag yet and a hand-edit would be + overwritten on the next sync. It flows in automatically when the SDK bumps + to a tag containing the fix (v0.7.2+). Tracked, not actioned. + - `omnigraph-cookbooks/docs/best-practices.md` `bearer_token_env` chain — + **RESOLVED** by omnigraph-cookbooks PR #26 (2026-06-21), which deleted + `docs/best-practices.md` as part of the 0.7 restructure; the stale chain + survives nowhere on `main`. ## Verification checklist (re-run on the next docs audit)