Phase 6: Integration, realistic workflows & documentation#20
Merged
Conversation
…umentation Bug fixes: - Fix API path mismatch: /api/components → /api/dynamic/components - Add EventBus Bridge to all e2e moduleTypeMaps Realistic workflow: - Add 10-component order processing pipeline example - Add 4 integration tests covering full pipeline execution Coverage improvements: - cmd/server: 20.3% → 70.9% (extract buildEngine/buildMux, add tests) - module: 78.2% → 80.9% (event processor, api handlers, health, middleware, namespace tests) Copilot SDK verification: - Add integration tests for tool handlers and provider selection - Add service-level provider fallback tests E2E testing: - Add 33 exploratory Playwright tests with 37 screenshots - Add 4 fixture YAML files for import/export tests Documentation: - Update README.md (CrisisTextLine/modular v1.11.11, Go 1.25, 30 module types) - Update ROADMAP.md (Phase 5 complete, Phase 6 added, coverage table updated) Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
- Use _ = for deferred os.Remove and tmpFile.Close return values - Replace nil with context.TODO() in Start/Stop test calls Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
intel352
added a commit
that referenced
this pull request
Feb 17, 2026
- Validate plugin name against directory traversal in CompositeRegistry.Install() - Add isSafePathComponent() to validate plugin name/version in HTTP handler - Validate planID in FileStateStore.GetPlan() to prevent path traversal Fixes code scanning alerts #20, #21, #22. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
/api/components→/api/dynamic/components(critical bug fix for UI-to-backend communication)Coverage
Test plan
go build ./...— clean buildgo test ./...— all tests passgo test -cover ./...— all coverage targets met🤖 Generated with Claude Code