status.yaml

status: active_implementation
started_at: "2026-03-04"
spec_files:
  - spec/phase2.8/SPEC_TABLE_RENDERING.md
  - spec/phase2.8/SPEC_OUTPUT_UPGRADE.md
  - spec/phase2.8/SPEC_CODE_STRUCTURE.md
  - spec/phase2.8/SPEC_TEST_MATRIX.md
  - spec/phase2.8/SPEC_TRACEABILITY.md
builder_progress:
  batch_1_commands_decomposition:
    status: completed
    completed_at: "2026-03-05"
    requirements: CST-001, CST-002, CST-003
    scenarios: TM-P28-001, TM-P28-002
    notes:
      - decomposed monolithic commands.rs (~3768 lines) into commands/ directory with 8 sub-modules
      - modules created: mod.rs, install.rs, reconcile.rs, diagnose.rs, plan_cmd.rs, config_ops.rs, remove.rs, update.rs, common.rs
      - public API unchanged via mod.rs re-exports
      - all 253 existing tests pass without modification
    quality_gate:
      fmt_check: passed
      clippy_workspace_deny_warnings: passed
      test_workspace: passed
      current_test_inventory: 253
  batch_2_table_infrastructure:
    status: completed
    completed_at: "2026-03-05"
    requirements: TBL-001, TBL-002
    scenarios: TM-P28-004, TM-P28-032, TM-P28-033
    notes:
      - added comfy-table = "7" to eden-skills-cli dependency set
      - added UiContext::table() factory with TTY UTF-8 and non-TTY ASCII border presets
      - added non-TTY fallback width (80) with dynamic content wrapping in table factory
      - added presentation-only path abbreviation helpers in ui.rs: abbreviate_home_path and abbreviate_repo_url
      - added integration tests in table_infra_tests.rs for dependency presence, table policy, and abbreviation helpers
    quality_gate:
      fmt_check: passed
      clippy_workspace_deny_warnings: passed
      test_workspace: passed
      current_test_inventory: 266
  batch_3_output_upgrade_category_a1:
    status: completed
    completed_at: "2026-03-05"
    requirements: OUP-001, OUP-002, OUP-004, OUP-013, OUP-014, OUP-015, OUP-016, OUP-017, OUP-019
    scenarios: TM-P28-012, TM-P28-013, TM-P28-014, TM-P28-015, TM-P28-016, TM-P28-017, TM-P28-021, TM-P28-022, TM-P28-026, TM-P28-027, TM-P28-028
    notes:
      - added output_upgrade_a1_tests.rs with TM-aligned coverage for apply/repair, plan, error hint/path, and warning formatting
      - upgraded apply and repair output in reconcile.rs to UiContext-driven human mode with Syncing/Safety action prefixes, per-target Install/Remove lines, styled Summary line, and Verification passed line
      - upgraded plan text output in plan_cmd.rs to include Plan header, palette-colored right-aligned action labels, unicode arrow separator, and up-to-date empty-state rendering
      - aligned error and warning display format to arrow-hint and two-space indented warning prefix via main.rs and common.rs helpers
      - updated legacy assertions in output_polish_tests.rs, exit_code_matrix.rs, and phase25_schema_tests.rs for the new human output format
    quality_gate:
      fmt_check: passed
      clippy_workspace_deny_warnings: passed
      test_workspace: passed
      current_test_inventory: 277
  batch_4_output_upgrade_category_a2:
    status: completed
    completed_at: "2026-03-05"
    requirements: OUP-003, OUP-005, OUP-006, OUP-007, OUP-018, OUP-020
    scenarios: TM-P28-018, TM-P28-019, TM-P28-020, TM-P28-023, TM-P28-024, TM-P28-025
    notes:
      - added output_upgrade_a2_tests.rs with TM-aligned assertions for doctor cards/header/summary table, init next-steps block, and install per-target/discovery output
      - upgraded doctor human output in diagnose.rs to UiContext action-prefix header, severity cards, dimmed arrow remediation, and conditional summary table when findings exceed 3
      - upgraded init output in config_ops.rs to show abbreviated-path success symbol and a 3-line Next steps guidance block with dimmed descriptions
      - upgraded install human output in install.rs to emit numbered Found discovery summary, per-target Install lines with unicode arrow paths, and final skills/agents/conflicts summary counts
      - aligned install warning paths to common styled warning formatter and refreshed legacy doctor text assertion in doctor_output.rs
    quality_gate:
      fmt_check: passed
      clippy_workspace_deny_warnings: passed
      test_workspace: passed
      current_test_inventory: 283
  batch_5_output_upgrade_category_b_tables:
    status: completed
    completed_at: "2026-03-05"
    requirements: TBL-003, TBL-004, TBL-005, TBL-006, TBL-007, OUP-008, OUP-009, OUP-010, OUP-011, OUP-012
    scenarios: TM-P28-005, TM-P28-006, TM-P28-007, TM-P28-008, TM-P28-009, TM-P28-010, TM-P28-011, TM-P28-029, TM-P28-030, TM-P28-031
    notes:
      - added output_upgrade_b_tests.rs with TM-aligned coverage for list/install/plan/update table output and non-TTY/color/json table constraints
      - upgraded list output in config_ops.rs to UiContext-driven `Skills` header plus `Skill | Mode | Source | Agents` table with source/path abbreviation and metadata-only suffix support
      - upgraded install dry-run and install --list output in install.rs to table-based human format (`Agent | Path | Mode` and `# | Name | Description`) with >8 discovery truncation footer
      - added plan table threshold branch in plan_cmd.rs so plans with more than 5 actions render `Action | Skill | Target | Mode` table while <=5 remains text format
      - upgraded update output in update.rs to action-prefixed registry results table (`Registry | Status | Detail`) plus final failed-count timing footer and styled warning reuse
      - refreshed legacy assertions in list_command.rs and phase2_commands.rs for new list/update/dry-run human output contracts
    quality_gate:
      fmt_check: passed
      clippy_workspace_deny_warnings: passed
      test_workspace: passed
      current_test_inventory: 293
  batch_6_doc_comments:
    status: completed
    completed_at: "2026-03-05"
    requirements: CST-004, CST-005, CST-006, CST-007, CST-008
    scenarios: TM-P28-003, TM-P28-034, TM-P28-035, TM-P28-036
    notes:
      - added //! module docs to all 9 commands/ sub-modules per SPEC_CODE_STRUCTURE.md 3.2
      - added /// doc comments with # Errors to all public command functions (install_async, apply_async, repair_async, doctor, plan, init, list, add, set, config_export, config_import, remove_many_async, update_async, remove, remove_async, apply, repair)
      - added /// doc comments to shared utilities in common.rs (resolve_config_path, load_config_with_context, ensure_git_available, ensure_docker_available_for_targets)
      - added //! crate doc to core lib.rs and //! module docs to 8 core modules (reactor, lock, adapter, source_format, discovery, config, plan, error)
      - added /// doc comments to core public types and functions (SkillReactor, run_phase_a, run_blocking, compute_lock_diff, read_lock_file, TargetAdapter, LocalAdapter, DockerAdapter, detect_install_source, discover_skills, validate_config, LoadedConfig, build_plan, Action, EdenError, ReactorError, AdapterError, RegistryError)
      - added //! module doc and /// on all public items in ui.rs (UiContext, UiSpinner, ColorWhen, StatusSymbol, configure_color_output, color_output_enabled, table, spinner, abbreviate_home_path, abbreviate_repo_url)
      - added //! crate doc to CLI lib.rs and /// on print_error in main.rs
      - added doc_coverage_tests.rs with 6 tests verifying //! and /// presence
      - followed Rust Best Practices skill Chapter 8 guidance throughout
    quality_gate:
      fmt_check: passed
      clippy_workspace_deny_warnings: passed
      test_workspace: passed
      current_test_inventory: 299
  batch_7_regression_closeout:
    status: completed
    completed_at: "2026-03-05"
    requirements: []
    scenarios: TM-P28-037, TM-P28-038, TM-P28-039, TM-P28-040
    notes:
      - full regression passed — 299 tests across all Phase 1/2/2.5/2.7/2.8 suites
      - JSON regression passed — all --json test assertions unmodified (doctor_json_contract, plan_json_contract, install_json, list_json, json_mode_never_renders_tables)
      - exit code regression passed — exit_code_matrix all 11 tests pass with 0/1/2/3 semantics unchanged
      - no hardcoded ANSI regression passed — zero \u{1b}[ matches in source code
      - updated SPEC_TRACEABILITY.md with TM-P28-037~040 results
      - updated README.md with Phase 2.8 completion status
      - updated docs/01-quickstart.md with new apply/doctor output examples
      - updated docs/05-safety-strict-and-exit-codes.md with arrow hint and path abbreviation
      - updated docs/README.md with Phase 2.8 spec references
    quality_gate:
      fmt_check: passed
      clippy_workspace_deny_warnings: passed
      test_workspace: passed
      current_test_inventory: 299