Skip to content

test(mcp): Phase 3 — fixture redaction contract#364

Merged
erishforG merged 1 commit into
release/1.0from
feat/auto-mcp-fixture-redaction-phase3
Jun 21, 2026
Merged

test(mcp): Phase 3 — fixture redaction contract#364
erishforG merged 1 commit into
release/1.0from
feat/auto-mcp-fixture-redaction-phase3

Conversation

@erishforG

@erishforG erishforG commented Jun 21, 2026

Copy link
Copy Markdown
Owner

무엇

MCP recording fixture README에 redaction contract와 recording checklist를 추가했습니다.

Refs #295. Claude Desktop / Cursor e2e fixture가 늘어나기 전에 token, local path, timestamp, network-generated ID가 저장되지 않도록 안전한 fixture 기준을 고정합니다. @erishforG

변경

  • tests/mcp/README.md에 redaction placeholder table 추가
  • token-shaped strings 및 authorization header 리뷰 기준 추가
  • fixture capture/review checklist 추가

다음 Phase 힌트

  • 실제 client recording fixture를 추가할 때 이 checklist를 기준으로 scrubbed fixture를 저장
  • 필요하면 fixture hygiene 검사 테스트를 별도 Phase로 추가

리스크

Low. 문서만 변경하며 CLI, MCP dispatcher, git/worktree 로직은 변경하지 않습니다.

롤백

tests/mcp/README.md의 redaction/checklist 섹션만 되돌리면 됩니다.

Test plan

  • cargo build --quiet
  • cargo fmt --check
  • cargo clippy --all-targets -- -D warnings
  • cargo test --quiet mcp::tests::stdio_recording_fixtures_match_dispatcher

Refs #295

Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com>
@erishforG erishforG added the auto-improvement Daily auto-generated improvement PR/issue label Jun 21, 2026
@erishforG erishforG merged commit 5285537 into release/1.0 Jun 21, 2026
12 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

auto-improvement Daily auto-generated improvement PR/issue

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant