Skip to content

test(mcp): Phase 4 — guard fixture redaction#365

Open
erishforG wants to merge 1 commit into
release/1.0from
feat/auto-295-phase4
Open

test(mcp): Phase 4 — guard fixture redaction#365
erishforG wants to merge 1 commit into
release/1.0from
feat/auto-295-phase4

Conversation

@erishforG

Copy link
Copy Markdown
Owner

무엇

MCP stdio recording fixtures에 대한 자동 redaction guard를 추가했습니다.

Refs #295. 최근 fixture redaction contract가 문서화되었고, 이번 Phase는 그 계약을 테스트로 고정해 토큰과 로컬 경로가 fixture에 커밋되는 위험을 줄입니다.

변경

  • stdio fixture 테스트가 공통 fixture reader helper를 사용하도록 정리
  • fixture 라인에서 GitHub token, bearer/auth header, common local path prefix를 거부하는 테스트 추가
  • tests/mcp README의 checklist를 새 테스트 패턴과 맞춤

다음 Phase 힌트

Claude Desktop/Cursor 실제 recording fixture를 추가할 때 redaction guard 범위를 확장하거나 fixture별 metadata validation을 추가할 수 있습니다.

리스크

Low. MCP 테스트와 문서만 변경하며 런타임 동작은 바꾸지 않습니다.

롤백

해당 커밋을 revert하면 기존 fixture matcher만 남습니다.

Test plan

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

@erishforG

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 22, 2026
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