Skip to content

test: regression coverage for HAR/SCS #1030 and recent renames - #1054

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

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

Conversation

@cursor

@cursor cursor Bot commented Oct 4, 2026

Copy link
Copy Markdown
Contributor

Description

Adds regression tests for production fixes merged since the last coverage automation run, focusing on #1030 (HAR/SCS query params, deep links, compact markers) plus small guards for #1043/#1045.

Risky behavior now covered

  • HAR list search: search_term must map to the search_term query key (not a silent search key) on registry/artifact/artifact_version/artifact_file list.
  • HAR artifact deep links: list pathParams must resolve {artifactIdentifier} in per-item openInHarness URLs.
  • HAR UPSTREAM create: registry create bodySchema documents required authType for UPSTREAM registries.
  • SCS deep links: artifact_security, scs_artifact_remediation, and scs_chain_of_custody must resolve {artifact} from artifact_id (not leave placeholders).
  • compact.ts: _summary, _message, _result marker fields from SCS list extractors survive default compaction.
  • Resource renames: retired observability_evaluation_rule is not registered; pr_comment execute surface is only resolve/unresolve after Remove redundant pr_comment set_status action #1043.

Test files added/updated

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

Why these tests materially reduce regression risk

These paths were fixed without accompanying tests in #1030; failures were silent (search filters ignored, broken UI deep links, stripped agent hints). The new tests lock the wire-format and deep-link contracts agents rely on during registry and supply-chain workflows.

Type of Change

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

Checklist

  • pnpm test passes (targeted: registries-har-search-deeplinks, scs, compact, observability-evaluations, pull-requests)
  • pnpm typecheck passes
  • pnpm build passes
  • pnpm standards:check passes
  • pnpm docs:check passes

Coding Standards (registry-driven MCP model)

N/A — test-only change.

Open in Web View Automation 

Add tests for HAR search_term query mapping, artifact deep links, UPSTREAM
authType schema docs, SCS artifact deep link placeholders, compact marker
fields, observability_eval rename, and pr_comment execute action surface.

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