LuCLI Phase 2: Service layer, generators, and MCP annotations

[bpamiri]

Summary

• Port 4 standalone services from CommandBox CLI to LuCLI (no WireBox): Templates, CodeGen, Scaffold, Analysis
• Rewire all 5 existing generators (model, controller, view, migration, scaffold) to use shared template files via service layer
• Add 3 new generators: route, test, property
• Improve server commands: TestBox JSON tree display, reload password auto-detection, enhanced info/routes output
• Add analyze and validate commands for code quality
• Annotate all public functions with hint for LuCLI MCP auto-discovery

Architecture

Services use constructor injection via init() — no WireBox container needed. getService() in Module.cfc lazy-loads with dependency wiring:

templates → new services.Templates(helpers, projectRoot, moduleRoot)
codegen   → new services.CodeGen(templateService, helpers, projectRoot)
scaffold  → new services.Scaffold(codeGenService, helpers, projectRoot)
analysis  → new services.Analysis(helpers, projectRoot)

Templates are shared from cli/src/templates/ with app/snippets/ overrides.

Test plan

• Verify lucli wheels generate model User name:string email:string creates model file — code review: parses properties, uses template service, writes to app/models/User.cfc
• Verify lucli wheels generate scaffold Post title body:text creates full CRUD — code review: orchestrates model + controller + views + migration + route (fixed pluralName bug in befe893)
• Verify lucli wheels generate route posts updates config/routes.cfm — code review: reads routes file, inserts .resources(), detects duplicates (fixed pluralName bug in befe893)
• Verify lucli wheels test parses TestBox JSON and displays tree — code review: handles both field name variants, displays bundle/suite/spec tree
• Verify lucli wheels analyze returns code quality metrics — code review: scans dirs, detects anti-patterns, returns structured metrics with health score
• Verify lucli mcp wheels exposes all annotated functions as MCP tools — verified: all 12 public functions have hint: docblock annotations
• Verify no WireBox references: grep -r "property inject=" cli/lucli/services/ returns empty — verified: grep returns no results

🤖 Generated with Claude Code

[github-actions]

Wheels Test Matrix

Engine	MySQL	PostgreSQL	SQL Server	H2	Oracle	SQLite
lucee5	✅	✅	✅	✅	✅	✅
lucee6	✅	✅	✅	✅	✅	✅
lucee7	✅	✅	✅	✅	✅	✅
adobe2018	✅	✅	✅	--	✅	--
adobe2021	✅	✅	✅	--	✅	✅
adobe2023	✅	✅	✅	--	✅	✅
adobe2025	✅	✅	✅	--	✅	✅
boxlang	✅	✅	✅	--	✅	✅

Results for commit c53442c.

[github-actions]

Wheels Test Results

    42 files  ±    0   3 780 suites  +84   25m 3s ⏱️ - 1m 3s
78 162 tests +1 260  77 774 ✅ +1 253  388 💤 +7  0 ❌ ±0 
80 808 runs  +1 260  80 420 ✅ +1 253  388 💤 +7  0 ❌ ±0 

Results for commit befe893. ± Comparison against base commit e5e2d59.

This pull request skips 7 tests.

adobe2021/sqlite :: Stress Testing for Race Conditions ‑ should handle concurrent model access with cache manipulation
adobe2023/sqlite :: Stress Testing for Race Conditions ‑ should handle concurrent model access with cache manipulation
adobe2025/sqlite :: Stress Testing for Race Conditions ‑ should handle concurrent model access with cache manipulation
boxlang/sqlite :: Stress Testing for Race Conditions ‑ should handle concurrent model access with cache manipulation
lucee5/sqlite :: Stress Testing for Race Conditions ‑ should handle concurrent model access with cache manipulation
lucee6/sqlite :: Stress Testing for Race Conditions ‑ should handle concurrent model access with cache manipulation
lucee7/sqlite :: Stress Testing for Race Conditions ‑ should handle concurrent model access with cache manipulation

♻️ This comment has been updated with latest results.

[bpamiri]

CI Failure Investigation

develop branch (commit e5e2d59) had 1 failing engine: adobe2025

Root cause: Adobe 2025 container crashed on startup 3 times, then timed out. No tests ran — this was a flaky infrastructure issue, not a test failure. The same engine version (2025.0.06+33156) passes on every other run.

This PR additionally hit:

• lucee7 (55 failures x 6 DBs): Lucee 7 snapshot auto-updated from 7.0.1+100 to 7.0.2+106 mid-day, which changed encodeForHTML() behavior (stopped hex-encoding /). Pinned to 7.0.1+100 to fix.
• adobe2025 (1 failure on SQLite): Flaky race condition test (raceconditionSpec > should handle concurrent model access with cache manipulation). Pre-existing, not caused by this PR.

The engine pinning commit should resolve the lucee7 failures. The adobe2025 SQLite race condition is a known flaky test.