Skip to content

♻️ Split TDD status plumbing#263

Merged
Robdel12 merged 1 commit into
rd/cli-audit-remaining-basefrom
rd/cli-audit-tdd-status
May 18, 2026
Merged

♻️ Split TDD status plumbing#263
Robdel12 merged 1 commit into
rd/cli-audit-remaining-basefrom
rd/cli-audit-tdd-status

Conversation

@Robdel12
Copy link
Copy Markdown
Contributor

@Robdel12 Robdel12 commented May 18, 2026

Why

The TDD/status/runtime path is the highest-risk CLI surface in this cleanup. It owns local daemon state, process lifecycle, build finalization, preview/status output, and baseline workflows. That work should not be hidden inside generic command cleanup because reviewers need to reason about process behavior and local state deliberately.

This PR groups the operational command slice together: doctor, preview, run, status, TDD, daemon management, and the runtime services that support them. It is larger than the other split PRs, but the files are related by one concern: keeping local visual testing workflows reliable.

What Changed

  • Hardened doctor, preview, run, status, tdd, and tdd-daemon behavior.
  • Cleaned up local build/server/test-runner service boundaries and removed stale server-manager service code.
  • Tightened TDD server registry, hotspot/region metadata, and daemon lifecycle behavior.
  • Added focused command, service, and TDD tests around process cleanup, daemon state, status rendering, preview behavior, and baseline workflows.

Verification

  • npm run build
  • node --test tests/commands/doctor.test.js tests/commands/preview.test.js tests/commands/run.test.js tests/commands/status.test.js tests/commands/tdd.test.js tests/commands/tdd-daemon.test.js tests/services/build-manager.test.js tests/services/static-report-generator.test.js tests/tdd/server-registry.test.js tests/tdd/tdd-service.test.js

Stack

8/10 in the CLI audit stack.

Base: rd/cli-audit-services-uploader
Head: rd/cli-audit-tdd-status

@Robdel12 Robdel12 force-pushed the rd/cli-audit-services-uploader branch from 4f542ac to 631f78f Compare May 18, 2026 19:31
Base automatically changed from rd/cli-audit-services-uploader to rd/cli-audit-remaining-base May 18, 2026 19:32
Move the ops commands and their supporting runtime services into the next stacked review branch.
@Robdel12 Robdel12 force-pushed the rd/cli-audit-tdd-status branch from 3f22a2a to e96043b Compare May 18, 2026 19:33
@Robdel12 Robdel12 merged commit 7084737 into rd/cli-audit-remaining-base May 18, 2026
@Robdel12 Robdel12 deleted the rd/cli-audit-tdd-status branch May 18, 2026 19:34
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