Skip to content

test: regression coverage for HAR/SCS #1030 fixes - #1050

Draft
cursor[bot] wants to merge 1 commit into
mainfrom
cursor/regression-test-coverage-automation-41be
Draft

cursor[bot] wants to merge 1 commit into
mainfrom
cursor/regression-test-coverage-automation-41be

Conversation

@cursor

@cursor cursor Bot commented Oct 3, 2026

Copy link
Copy Markdown
Contributor

Description

Adds regression tests for production fixes merged in #1030 where list search_term was mapped to a bogus search query key, HAR artifact deep links left {artifactIdentifier} unresolved, SCS deep link templates used {artifactId} instead of {artifact}, and compactItems stripped SCS _summary / _message / _result markers.

Risky behavior now covered

  • HAR list filters: registry, artifact, artifact_version, and artifact_file list ops must send search_term to the HAR API (not search), or search filters silently do nothing.
  • HAR artifact deep links: artifact list must map artifact_id → artifactIdentifier so openInHarness URLs are usable.
  • UPSTREAM registry create: harness_schema docs for registry create must document required config.authType for UPSTREAM registries.
  • SCS deep links: artifact_security, scs_artifact_remediation, and scs_chain_of_custody templates and get responses must resolve {artifact} without literal placeholders.
  • List compaction: compactItems must preserve _summary, _message, and _result fields attached by SCS list extractors.

Test files added/updated

  • tests/registry/registries-har-search-deeplinks.test.ts (new)
  • tests/registry/scs.test.ts
  • tests/utils/compact.test.ts

Why these tests materially reduce regression risk

These bugs were discovered during live QA: incorrect query-param keys and pathParam/deep-link mismatches fail silently (empty results or broken UI links) rather than with obvious API errors. The tests assert the exact HTTP query keys, pathParam mappings, schema documentation strings, and compaction whitelist behavior so a revert of #1030 would fail CI immediately.

Validation

  • pnpm test tests/registry/registries-har-search-deeplinks.test.ts tests/registry/scs.test.ts tests/utils/compact.test.ts — 315 tests passed

Type of Change

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

Checklist

  • pnpm test passes (targeted suites above)
  • pnpm typecheck passes
  • pnpm build passes
  • pnpm standards:check passes
  • pnpm docs:check passes
Open in Web View Automation 

Lock search_term query mapping, artifact deep links, SCS {artifact}
placeholder alignment, and compact marker field preservation.

Co-authored-by: Rohan Gupta <thisrohangupta@users.noreply.github.com>
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