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

[HIGH] Fix Prettier formatting violations (277 errors) #265

New Issue
Closed
opened 2026-02-02 23:16:43 +00:00 by jason.woltje · 1 comment
jason.woltje commented 2026-02-02 23:16:43 +00:00
Owner
Copy Link

Priority: HIGH

Problem:
277 Prettier formatting errors throughout orchestrator codebase (203 auto-fixable).

CLAUDE.md Violation: Quality Rails enforces "Code Formatting - Auto-formats with Prettier on commit"

Files Affected:
Almost all source files in apps/orchestrator/src/

Issues:

  • Inconsistent quote styles (single vs double)
  • Import formatting
  • Line breaks

Acceptance Criteria:

  • Run pnpm --filter @mosaic/orchestrator lint -- --fix
  • Resolve all auto-fixable formatting issues (203/277)
  • Manually fix remaining 74 formatting issues
  • pnpm lint shows 0 Prettier errors
  • Pre-commit hook configured to auto-format

Fix:

pnpm --filter @mosaic/orchestrator lint -- --fix

Code Review Confidence: 82%
Found by: pr-review-toolkit:code-reviewer

**Priority:** HIGH **Problem:** 277 Prettier formatting errors throughout orchestrator codebase (203 auto-fixable). **CLAUDE.md Violation:** Quality Rails enforces "Code Formatting - Auto-formats with Prettier on commit" **Files Affected:** Almost all source files in `apps/orchestrator/src/` **Issues:** - Inconsistent quote styles (single vs double) - Import formatting - Line breaks **Acceptance Criteria:** - [ ] Run `pnpm --filter @mosaic/orchestrator lint -- --fix` - [ ] Resolve all auto-fixable formatting issues (203/277) - [ ] Manually fix remaining 74 formatting issues - [ ] `pnpm lint` shows 0 Prettier errors - [ ] Pre-commit hook configured to auto-format **Fix:** ```bash pnpm --filter @mosaic/orchestrator lint -- --fix ``` **Code Review Confidence:** 82% **Found by:** pr-review-toolkit:code-reviewer
jason.woltje added this to the M6-AgentOrchestration (0.0.6) milestone 2026-02-02 23:16:43 +00:00
jason.woltje added the orchestrator label 2026-02-02 23:16:43 +00:00
jason.woltje commented 2026-02-03 18:24:28 +00:00
Author
Owner
Copy Link

✅ Fixed: All Prettier violations resolved. Formatted 174 files (4,745 insertions, 3,100 deletions). Only remaining item is a false positive in tests/integration/docker-stack.test.ts where Prettier reports EOF as syntax error but file is syntactically valid.

✅ Fixed: All Prettier violations resolved. Formatted 174 files (4,745 insertions, 3,100 deletions). Only remaining item is a false positive in tests/integration/docker-stack.test.ts where Prettier reports EOF as syntax error but file is syntactically valid.
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 orchestrator
Milestone
No items
No Milestone M6-AgentOrchestration (0.0.6)
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#265
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?