Skip to content

chore: update CLAUDE.md and UnitTests.xctestplan for 37 new features#122

Open
StevenLi-phoenix wants to merge 1 commit intomainfrom
chore/update-project-docs
Open

chore: update CLAUDE.md and UnitTests.xctestplan for 37 new features#122
StevenLi-phoenix wants to merge 1 commit intomainfrom
chore/update-project-docs

Conversation

@StevenLi-phoenix
Copy link
Copy Markdown
Owner

Summary

  • CLAUDE.md: Document all new services added across PRs feat: auto-generate action items from transcripts #57feat: system audio capture via ScreenCaptureKit #120 (SystemAudioCaptureService, SpotlightIndexer, HandoffService, FocusModeService, DiagnosticsCollector, ModelRecommendation, MeetingRecapFormatter, WaveformExtractor), updated architecture section, test counts (~100 files, ~1000+ tests), default model (qwen3.5-9b-mlx), entitlements (Screen Recording for ScreenCaptureKit), Settings tab count (5 tabs)
  • UnitTests.xctestplan: Add 36 new non-serialized test suites (58 total, up from 22). Excludes 10 serialized suites (ActionItemMarkdownFormatterTests, ActionItemParserTests, AutoExportTests, FocusModeServiceTests, MatchedGeometryTests, MeetingTemplateTests, SessionTrashTests, StructuredSummaryTests, SummarizerStructuredTests, TokenCountingTests)

Test plan

  • UnitTests.xctestplan is valid JSON
  • All serialized suites excluded from UnitTests plan
  • CLAUDE.md architecture section reflects current codebase

- CLAUDE.md: document new services (SystemAudioCaptureService,
  SpotlightIndexer, HandoffService, FocusModeService, DiagnosticsCollector,
  ModelRecommendation, MeetingRecapFormatter, WaveformExtractor), updated
  architecture section, test counts, default model, entitlements
- UnitTests.xctestplan: add 36 new non-serialized test suites (58 total,
  up from 22); excludes serialized suites for fast local iteration
@claude
Copy link
Copy Markdown

claude bot commented Mar 28, 2026

✅ Claude Code Review: APPROVE

Documentation-only and test-plan update accurately reflects new services added in prior PRs and correctly extends UnitTests.xctestplan with non-serialized suites following the project's two-tier test plan pattern.

@github-actions
Copy link
Copy Markdown
Contributor

⚠️ Auto-merge skipped: Tests, unchecked checkboxes, merge conflicts, or merge step failed. Manual review required.

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