Skip to content

chore: remove legacy snap test infrastructure after PTY migration #2136

Description

@jong-kyung

Summary

Remove the obsolete infrastructure, configuration, and active documentation for the legacy snapshot suites after the PTY migration tracked in #2116 is complete.

Context

The legacy suites under packages/cli/snap-tests/ and packages/cli/snap-tests-global/ will no longer be needed after the migration.

#1723 added the legacy paths to the VS Code Vitest exclusion, and #2135 retains them while adding the PTY snapshot path. Active repository guidance such as AGENTS.md, CONTRIBUTING.md, and the PTY snapshot README also still documents the legacy suites.

Done when

  • Remove the legacy path from vitest.configSearchPatternExclude without changing the PTY snapshot exclusion.
  • Remove obsolete legacy test scripts, CI wiring, configuration exclusions, and migration-only tooling.
  • Update active repository documentation and agent guidance to reference only the PTY snapshot suite.
  • Verify that the PTY suite is the only supported CLI snapshot test path.

Metadata

Metadata

Assignees

Labels

No labels
No labels

Type

Fields

Priority

None yet

Effort

None yet

Target date

None yet

Start date

None yet

Projects

No projects

Milestone

No milestone

Relationships

None yet

Development

No branches or pull requests

Issue actions