test: cover dbops authoring flow and entity schema cache gaps

[cursor]

Risky behavior now covered

DBOPS LLM authoring flow (PR #309)

• database_snapshot_object list — query param mapping (object_type → objectType, size → limit) and snapshot-names path construction
• database_snapshot_object get — object_names validation (missing, empty, non-array) and default object_type: "Table"
• database_default_authoring_instance get — path and response contract for auto-selecting an authoring instance
• database_llm_authoring_pipeline get — dbSchema/dbInstance query params for Accept & Commit pipeline resolution
• database_execute_llm_authoring_pipeline run — read-only mode blocks low_write execute; bodyBuilder omits empty runtime_inputs, accepts camelCase branches, enforces mutual exclusion; skipScopeBodyInjection metadata

Entity schema cache (PR #396)

• preloadBundledEntitySchemas — warms runtime cache on account match, skips warm on mismatch, does not overwrite existing entries
• Scope validation — org scope without org_id, project scope without project_id (fetcher + harness_schema tool)
• Org-scope cache isolation — distinct org IDs produce separate cache entries (prevents cross-org schema bleed)
• Live navigation helpers — getResourceDefinitions, getRootDefinition, navigateEntitySchemaPath, getEntitySchemaSummary bundled vs live hints
• extractLiveSchema — yamlSchema and nested data.yaml_schema envelopes; throws on non-schema responses
• buildBundledSchemaKey and org-level bundledSnapshotMatchesScope rejection

Test files added/updated

File	Change
tests/registry/dbops.test.ts	+6 describe blocks (snapshot list/get, authoring GETs, read-only gate)
tests/registry/toolsets/dbops.test.ts	+4 bodyBuilder/metadata edge cases
tests/tools/entity-schema-bundled.test.ts	scope validation + preloadBundledEntitySchemas warm path
tests/tools/entity-schema-cache-keys.test.ts	buildBundledSchemaKey, org scope gate
tests/tools/harness-schema-tool.test.ts	org/project scope validation, org cache isolation
tests/tools/entity-schema-live.test.ts	new — navigation helpers + extractLiveSchema shapes

Why these tests materially reduce regression risk

• DBOPS Accept & Commit is a billing-adjacent write path; incorrect body normalization or read-only bypass would cause silent API failures or unintended mutations.
• Entity schema cache bugs cause cross-tenant/org schema bleed — agents would receive wrong field definitions when creating Harness resources.
• Scope validation gaps previously only had partial coverage (project without org_id); org and project_id guards are now enforced at both fetcher and tool layers.

Validation

pnpm test tests/registry/dbops.test.ts tests/registry/toolsets/dbops.test.ts \
  tests/tools/entity-schema-bundled.test.ts tests/tools/entity-schema-cache-keys.test.ts \
  tests/tools/harness-schema-tool.test.ts tests/tools/entity-schema-live.test.ts

116 tests passed (6 files).

  

[CLAassistant]

Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you all sign our Contributor License Agreement before we can accept your contribution.
1 out of 2 committers have signed the CLA.

✅ thisrohangupta
❌ cursoragent
_{You have signed the CLA already but the status is still pending? Let us recheck it.}

[thisrohangupta]

Closing as stale: merge conflicts with main and work superseded by merged PRs (#447, #446, #449, #469, #440, #398, etc.) or duplicate agent-generated attempts. Reopen from a fresh branch if still needed.