mosaic/stack
1
0
Fork 0
Code Issues 10 Pull Requests Actions Packages Projects Releases 2 Wiki Activity

Phase 3: Medium Priority - Code Quality + Infrastructure #339

New Issue
Closed
opened 2026-02-05 21:12:52 +00:00 by jason.woltje · 0 comments
jason.woltje commented 2026-02-05 21:12:52 +00:00
Owner
Copy Link

Findings (Medium Severity)

Code Quality:

  • CQ-ORCH-5: TOCTOU race in state transitions
  • CQ-WEB-7: KanbanBoard no UI revert on error
  • CQ-ORCH-2: Race condition - dual state management
  • CQ-ORCH-4: No timeout cleanup for AbortController
  • CQ-API-4: Missing cleanup for Redis event listeners
  • CQ-WEB-2: Missing dependency in FilterBar useEffect
  • CQ-WEB-3: Race condition in LinkAutocomplete
  • CQ-WEB-6: Memory leak in Chat component timers
  • CQ-ORCH-7: Docker force remove loses data
  • CQ-ORCH-8: KillAllAgents doesn't distinguish error types
  • CQ-API-3: Activity logging re-throws (defeats fire-and-forget)

Security - Medium:

  • SEC-API-12 through SEC-API-24 (13 findings)
  • SEC-WEB-12 through SEC-WEB-25 (14 findings)
  • SEC-ORCH-16 through SEC-ORCH-27 (12 findings)

Acceptance Criteria

  • All medium-priority findings remediated
  • Quality gates passing
  • No new regressions
## Findings (Medium Severity) **Code Quality:** - CQ-ORCH-5: TOCTOU race in state transitions - CQ-WEB-7: KanbanBoard no UI revert on error - CQ-ORCH-2: Race condition - dual state management - CQ-ORCH-4: No timeout cleanup for AbortController - CQ-API-4: Missing cleanup for Redis event listeners - CQ-WEB-2: Missing dependency in FilterBar useEffect - CQ-WEB-3: Race condition in LinkAutocomplete - CQ-WEB-6: Memory leak in Chat component timers - CQ-ORCH-7: Docker force remove loses data - CQ-ORCH-8: KillAllAgents doesn't distinguish error types - CQ-API-3: Activity logging re-throws (defeats fire-and-forget) **Security - Medium:** - SEC-API-12 through SEC-API-24 (13 findings) - SEC-WEB-12 through SEC-WEB-25 (14 findings) - SEC-ORCH-16 through SEC-ORCH-27 (12 findings) ## Acceptance Criteria - [ ] All medium-priority findings remediated - [ ] Quality gates passing - [ ] No new regressions
jason.woltje added this to the M6-AgentOrchestration-Fixes milestone 2026-02-05 22:09:45 +00:00
Sign in to join this conversation.
No Branch/Tag Specified
Branches Tags
Labels
Clear labels
ai
AI/LLM integration
 api
Backend/API work
 api
auth
Authentication
 database
database
Database/schema work
 devops
Docker/deployment
 docs
Documentation
 frontend
graph
knowledge-module
migration
Data migration
 orchestrator
Orchestrator service (apps/orchestrator/)
 p0
Critical priority
 p1
High priority
 p2
Medium priority
 p3
Low priority
 performance
Performance work
 phase-1
phase-2
phase-3
phase-4
phase-5
plugin
MoltBot plugin work
 search
security
Security-related
 setup
Initial setup
 testing
Testing/QA
 web
Frontend work
No Label
Milestone
No items
No Milestone M6-AgentOrchestration-Fixes
Projects
Clear projects
No project
Assignees
Clear assignees
jason.woltje
No Assignees
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: mosaic/stack#339
Delete Branch "%!s()"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?