Skip to content

test: lock DB DevOps reuse field in bundled pipeline/template schemas - #825

Draft
cursor[bot] wants to merge 2 commits into
mainfrom
cursor/missing-test-coverage-4249
Draft

test: lock DB DevOps reuse field in bundled pipeline/template schemas#825
cursor[bot] wants to merge 2 commits into
mainfrom
cursor/missing-test-coverage-4249

Conversation

@cursor

@cursor cursor Bot commented Aug 15, 2026

Copy link
Copy Markdown
Contributor

Description

Adds schema bundle contract coverage for the upstream reuse field introduced in the auto-sync harness schemas PR (#823).

Triggered by: chore/sync-schemas (#823) — upstream added reuse on DB DevOps step definitions (Flyway, Liquibase, DB schema apply/rollback, etc.). Without a contract test, a future sync or manual edit could drop the field silently while harness_schema still serves stale bundled YAML.

Risky behavior now covered

  • Presence of reuse on seven DB DevOps step definitions in both v0 pipeline and template bundled schemas
  • reuse accepts literal strings (non-expression) via not: { pattern: "^<\+.*>.*$" }
  • reuse accepts Harness input expressions (<+input>…) with the expected modifier vocabulary (allowedValues, selectOneFrom, and selectManyFrom where upstream defines it)

Test files added/updated

  • tests/schemas/schema-bundle-contract.test.ts

Why this reduces regression risk

  • harness_schema and entity validation read these bundled snapshots; missing reuse would mislead agents authoring DB DevOps pipelines
  • Follows the established upstream-sync contract pattern (K8s rollback steps, agent runtime deploy steps, etc.) so CI fails if synced schemas lose newly added fields

Type of Change

  • Bug fix
  • New feature
  • Refactor
  • Documentation
  • Other (test coverage)

Checklist

  • pnpm test passes (tests/schemas/schema-bundle-contract.test.ts)
  • pnpm build passes
  • pnpm typecheck passes
  • pnpm standards:check passes
  • pnpm docs:check passes (not applicable — no registry/tool count changes)

Coding Standards (registry-driven MCP model)

Not applicable — test-only change.

Open in Web View Automation 

thisrohangupta and others added 2 commits August 15, 2026 00:15
Add schema bundle contract assertions for the upstream reuse property
on DB DevOps step definitions synced in chore/sync-schemas (#823).

Co-authored-by: Rohan Gupta <thisrohangupta@users.noreply.github.com>
@CLAassistant

Copy link
Copy Markdown

CLA assistant check
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.

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.

3 participants