savepoint 1.0.1 → 1.0.3
This diff represents the content of publicly available package versions that have been released to one of the supported registries. The information contained in this diff is provided for informational purposes only and reflects changes between package versions as they appear in their respective public registries.
- package/.claude/settings.local.json +15 -1
- package/.golangci.yml +11 -0
- package/.savepoint/Design.md +52 -46
- package/.savepoint/releases/v1/epics/E01-go-setup/tasks/T001-init-module.md +1 -1
- package/.savepoint/releases/v1/epics/E03-board-tui-core/tasks/T005-layout.md +1 -1
- package/.savepoint/releases/v1/epics/E04-board-components/tasks/T002-card.md +1 -1
- package/.savepoint/releases/v1/epics/E04-board-components/tasks/T006-help-overlay.md +1 -1
- package/.savepoint/releases/v1/epics/E06-atari-noir-layout/{Design.md → E06-Detail.md} +5 -3
- package/.savepoint/releases/v1/epics/E06-atari-noir-layout/tasks/T002-header-and-dividers.md +1 -1
- package/.savepoint/releases/v1/epics/E06-atari-noir-layout/tasks/T003-footer-status-bar.md +1 -1
- package/.savepoint/releases/v1/epics/E06-atari-noir-layout/tasks/T004-component-refinement.md +1 -1
- package/.savepoint/releases/v1/epics/E06-atari-noir-layout/tasks/T010-auto-refresh-watcher.md +2 -0
- package/.savepoint/releases/v1.1/epics/E01-tui-optimisation/{Design.md → E01-Detail.md} +9 -1
- package/.savepoint/releases/v1.1/epics/E01-tui-optimisation/tasks/{T007-next-activity-header.md → T001-next-activity-header.md} +13 -12
- package/.savepoint/releases/v1.1/epics/E01-tui-optimisation/tasks/T002-rename-epic-design-files.md +9 -9
- package/.savepoint/releases/v1.1/epics/E01-tui-optimisation/tasks/T003-rename-release-prd.md +2 -2
- package/.savepoint/releases/v1.1/epics/E01-tui-optimisation/tasks/T004-update-instruction-files.md +13 -12
- package/.savepoint/releases/v1.1/epics/E01-tui-optimisation/tasks/T005-update-cross-references.md +14 -13
- package/.savepoint/releases/v1.1/epics/E01-tui-optimisation/tasks/T006-column-and-detail-scrolling.md +25 -15
- package/.savepoint/releases/v1.1/epics/E01-tui-optimisation/tasks/T007-column-focus-border-stability.md +57 -0
- package/.savepoint/releases/v1.1/epics/E02-cross-platform-compatibility/E02-Audit.md +124 -0
- package/.savepoint/releases/v1.1/epics/E02-cross-platform-compatibility/{Design.md → E02-Detail.md} +12 -3
- package/.savepoint/releases/v1.1/epics/E02-cross-platform-compatibility/tasks/T001-fix-makefile.md +11 -8
- package/.savepoint/releases/v1.1/epics/E02-cross-platform-compatibility/tasks/T002-linux-build-target.md +12 -7
- package/.savepoint/releases/v1.1/epics/E02-cross-platform-compatibility/tasks/T003-macos-build-target.md +9 -5
- package/.savepoint/releases/v1.1/epics/E02-cross-platform-compatibility/tasks/T004-smoke-tests-and-artifacts.md +30 -9
- package/.savepoint/releases/v1.1/epics/E03-ui-visual-refinement/E03-Audit.md +195 -0
- package/.savepoint/releases/v1.1/epics/E03-ui-visual-refinement/E03-Detail.md +45 -0
- package/.savepoint/releases/v1.1/epics/E03-ui-visual-refinement/tasks/T001-border-resize-fix.md +40 -0
- package/.savepoint/releases/v1.1/epics/E03-ui-visual-refinement/tasks/T002-next-activity-below-header.md +64 -0
- package/.savepoint/releases/v1.1/epics/E03-ui-visual-refinement/tasks/T003-checkbox-rendering-fix.md +56 -0
- package/.savepoint/releases/v1.1/epics/E03-ui-visual-refinement/tasks/T005-unify-status-glyphs.md +65 -0
- package/.savepoint/releases/v1.1/epics/E03-ui-visual-refinement/tasks/T006-forced-256-color-profile.md +36 -0
- package/.savepoint/releases/v1.1/epics/E04-epic-navigation/E04-Audit.md +167 -0
- package/.savepoint/releases/v1.1/epics/E04-epic-navigation/E04-Detail.md +51 -0
- package/.savepoint/releases/v1.1/epics/E04-epic-navigation/tasks/T001-sidebar-focusable-navigation.md +65 -0
- package/.savepoint/releases/v1.1/epics/E04-epic-navigation/tasks/T002-epic-detail-overlay.md +73 -0
- package/.savepoint/releases/v1.1/epics/E04-epic-navigation/tasks/T003-epic-status-glyphs.md +73 -0
- package/.savepoint/releases/v1.1/epics/E05-tasking-permissions/E05-Audit.md +237 -0
- package/.savepoint/releases/v1.1/epics/E05-tasking-permissions/E05-Detail.md +54 -0
- package/.savepoint/releases/v1.1/epics/E05-tasking-permissions/tasks/T001-update-agents-md.md +45 -0
- package/.savepoint/releases/v1.1/epics/E05-tasking-permissions/tasks/T002-update-router-md.md +40 -0
- package/.savepoint/releases/v1.1/epics/E05-tasking-permissions/tasks/T003-update-design-md.md +47 -0
- package/.savepoint/releases/v1.1/epics/E05-tasking-permissions/tasks/T004-implement-m-hotkey.md +98 -0
- package/.savepoint/releases/v1.1/epics/E05-tasking-permissions/tasks/T005-update-help-overlay.md +33 -0
- package/.savepoint/releases/v1.1/epics/E05-tasking-permissions/tasks/T006-tests-and-quality-gates.md +62 -0
- package/.savepoint/releases/v1.1/epics/E06-audit-command/E06-Audit.md +56 -0
- package/.savepoint/releases/v1.1/epics/E06-audit-command/E06-Detail.md +63 -0
- package/.savepoint/releases/v1.1/epics/E06-audit-command/tasks/T005-proposals.md +44 -0
- package/.savepoint/releases/v1.1/epics/E06-audit-command/tasks/T007-apply-close.md +35 -0
- package/.savepoint/releases/v1.1/epics/E06-audit-command/tasks/T009-integration.md +40 -0
- package/.savepoint/releases/v1.1/epics/E06-audit-command/tasks/T010-audit-file-migration.md +45 -0
- package/.savepoint/releases/v1.1/epics/E06-audit-command/tasks/T011-model-tab-state.md +26 -0
- package/.savepoint/releases/v1.1/epics/E06-audit-command/tasks/T012-epic-audit-render.md +33 -0
- package/.savepoint/releases/v1.1/epics/E06-audit-command/tasks/T013-handle-tab-keys.md +34 -0
- package/.savepoint/releases/v1.1/epics/E06-audit-command/tasks/T014-tab-indicator.md +33 -0
- package/.savepoint/releases/v1.1/epics/E07-init-command/E07-Audit.md +336 -0
- package/.savepoint/releases/v1.1/epics/E07-init-command/E07-Detail.md +61 -0
- package/.savepoint/releases/v1.1/epics/E07-init-command/tasks/T001-cli-entrypoint.md +37 -0
- package/.savepoint/releases/v1.1/epics/E07-init-command/tasks/T002-target-validation.md +28 -0
- package/.savepoint/releases/v1.1/epics/E07-init-command/tasks/T003-scaffold-writer.md +46 -0
- package/.savepoint/releases/v1.1/epics/E07-init-command/tasks/T004-atomic-writes.md +27 -0
- package/.savepoint/releases/v1.1/epics/E07-init-command/tasks/T005-magic-prompt.md +25 -0
- package/.savepoint/releases/v1.1/epics/E07-init-command/tasks/T006-clipboard.md +26 -0
- package/.savepoint/releases/v1.1/epics/E07-init-command/tasks/T007-integration-test.md +26 -0
- package/.savepoint/releases/v1.1/epics/E08-board-command/E08-Audit.md +333 -0
- package/.savepoint/releases/v1.1/epics/E08-board-command/E08-Detail.md +68 -0
- package/.savepoint/releases/v1.1/epics/E08-board-command/tasks/T001-cli-entrypoint.md +26 -0
- package/.savepoint/releases/v1.1/epics/E08-board-command/tasks/T002-non-tty-fallback.md +27 -0
- package/.savepoint/releases/v1.1/epics/E08-board-command/tasks/T003-tui-app-shell.md +28 -0
- package/.savepoint/releases/v1.1/epics/E08-board-command/tasks/T004-board-model.md +29 -0
- package/.savepoint/releases/v1.1/epics/E08-board-command/tasks/T005-detail-pane.md +27 -0
- package/.savepoint/releases/v1.1/epics/E08-board-command/tasks/T006-status-transitions.md +29 -0
- package/.savepoint/releases/v1.1/epics/E08-board-command/tasks/T007-theme-fallbacks.md +29 -0
- package/.savepoint/releases/v1.1/epics/E08-board-command/tasks/T008-integration-test.md +27 -0
- package/.savepoint/releases/v1.1/epics/E09-doctor-command/E09-Audit.md +207 -0
- package/.savepoint/releases/v1.1/epics/E09-doctor-command/E09-Detail.md +65 -0
- package/.savepoint/releases/v1.1/epics/E09-doctor-command/tasks/T001-cli-entrypoint.md +24 -0
- package/.savepoint/releases/v1.1/epics/E09-doctor-command/tasks/T002-config-router-validation.md +28 -0
- package/.savepoint/releases/v1.1/epics/E09-doctor-command/tasks/T003-structure-checks.md +29 -0
- package/.savepoint/releases/v1.1/epics/E09-doctor-command/tasks/T004-dependency-checks.md +27 -0
- package/.savepoint/releases/v1.1/epics/E09-doctor-command/tasks/T005-audit-orphan-checks.md +28 -0
- package/.savepoint/releases/v1.1/epics/E09-doctor-command/tasks/T006-quality-gates-report.md +31 -0
- package/.savepoint/releases/v1.1/epics/E11-board-refresh-fix/E11-Detail.md +36 -0
- package/.savepoint/releases/v1.1/epics/E11-board-refresh-fix/tasks/T001-debug-logging.md +25 -0
- package/.savepoint/releases/v1.1/epics/E11-board-refresh-fix/tasks/T002-increase-debounce.md +21 -0
- package/.savepoint/releases/v1.1/epics/E11-board-refresh-fix/tasks/T003-error-handling.md +22 -0
- package/.savepoint/releases/v1.1/epics/E11-board-refresh-fix/tasks/T004-test-verify.md +29 -0
- package/.savepoint/releases/v1.1/epics/E12-validation-fix/E12-Audit.md +444 -0
- package/.savepoint/releases/v1.1/epics/E12-validation-fix/E12-Detail.md +45 -0
- package/.savepoint/releases/v1.1/epics/E12-validation-fix/tasks/T001-default-phase.md +35 -0
- package/.savepoint/releases/v1.1/epics/E12-validation-fix/tasks/T002-default-status.md +19 -0
- package/.savepoint/releases/v1.1/epics/E12-validation-fix/tasks/T003-better-errors.md +29 -0
- package/.savepoint/releases/v1.1/epics/E12-validation-fix/tasks/T004-validate-on-write.md +25 -0
- package/.savepoint/releases/v1.1/epics/E12-validation-fix/tasks/T005-tests.md +37 -0
- package/.savepoint/releases/v1.1/epics/E13-audit-remediation/E13-Audit.md +118 -0
- package/.savepoint/releases/v1.1/epics/E13-audit-remediation/E13-Detail.md +73 -0
- package/.savepoint/releases/v1.1/epics/E13-audit-remediation/tasks/T001-safe-cleanup.md +66 -0
- package/.savepoint/releases/v1.1/epics/E13-audit-remediation/tasks/T002-bug-fixes.md +35 -0
- package/.savepoint/releases/v1.1/epics/E13-audit-remediation/tasks/T003-centralize-duplication.md +60 -0
- package/.savepoint/releases/v1.1/epics/E13-audit-remediation/tasks/T004-infrastructure.md +33 -0
- package/.savepoint/releases/v1.1/epics/E13-audit-remediation/tasks/T005-decompose-update.md +37 -0
- package/.savepoint/releases/v1.1/epics/E13-audit-remediation/tasks/T006-async-io.md +40 -0
- package/.savepoint/releases/v1.1/epics/E13-audit-remediation/tasks/T007-test-coverage.md +37 -0
- package/.savepoint/releases/v1.1/epics/E14-structural-improvements/E14-Audit.md +267 -0
- package/.savepoint/releases/v1.1/epics/E14-structural-improvements/E14-Detail.md +54 -0
- package/.savepoint/releases/v1.1/epics/E14-structural-improvements/tasks/T001-group-model.md +39 -0
- package/.savepoint/releases/v1.1/epics/E14-structural-improvements/tasks/T002-data-interfaces.md +42 -0
- package/.savepoint/releases/v1.1/epics/E14-structural-improvements/tasks/T003-discover-orphans.md +33 -0
- package/.savepoint/releases/v1.1/epics/E14-structural-improvements/tasks/T004-epic-panel-headings.md +35 -0
- package/.savepoint/releases/v1.1/epics/E14-structural-improvements/tasks/T005-shell-tokenization.md +27 -0
- package/.savepoint/releases/v1.1/epics/E14-structural-improvements/tasks/T006-unify-enums.md +29 -0
- package/.savepoint/releases/v1.1/epics/E14-structural-improvements/tasks/T007-testutil-package.md +28 -0
- package/.savepoint/releases/v1.1/epics/E15-hardening/E15-Detail.md +43 -0
- package/.savepoint/releases/v1.1/epics/E15-hardening/tasks/T001-benchmarks.md +31 -0
- package/.savepoint/releases/v1.1/epics/E15-hardening/tasks/T002-fuzz-targets.md +28 -0
- package/.savepoint/releases/v1.1/epics/E15-hardening/tasks/T003-debug-flag.md +30 -0
- package/.savepoint/releases/v1.1/epics/E15-hardening/tasks/T004-dist-checksums.md +27 -0
- package/.savepoint/releases/v1.1/epics/E15-hardening/tasks/T005-windows-targets.md +28 -0
- package/.savepoint/releases/v1.1/epics/E15-hardening/tasks/T006-abbreviation-splitting.md +26 -0
- package/.savepoint/releases/v1.1/epics/E15-hardening/tasks/T007-root-test-allowlist.md +28 -0
- package/.savepoint/releases/v1.1/epics/_archived/T001-cli-entrypoint.md +25 -0
- package/.savepoint/releases/v1.1/epics/_archived/T002-quality-gates.md +27 -0
- package/.savepoint/releases/v1.1/epics/_archived/T003-snapshot.md +27 -0
- package/.savepoint/releases/v1.1/epics/_archived/T004-ai-reconcile.md +29 -0
- package/.savepoint/releases/v1.1/epics/_archived/T006-tui-review.md +31 -0
- package/.savepoint/releases/v1.1/epics/_archived/T008-skip-handling.md +34 -0
- package/.savepoint/releases/v1.1/v1.1-PRD.md +139 -0
- package/.savepoint/router.md +29 -108
- package/AGENTS.md +69 -111
- package/Makefile +19 -3
- package/README.md +6 -6
- package/agent-skills/savepoint-audit/SKILL.md +87 -35
- package/agent-skills/savepoint-build-task/SKILL.md +9 -4
- package/agent-skills/savepoint-create-plan/SKILL.md +10 -5
- package/agent-skills/savepoint-create-task/SKILL.md +44 -31
- package/agent-skills/savepoint-draft-prd/SKILL.md +8 -3
- package/agent-skills/savepoint-system-design/SKILL.md +8 -3
- package/agent_skills_test.go +91 -0
- package/cmd/board.go +59 -0
- package/cmd/board_test.go +137 -0
- package/cmd/doctor.go +53 -0
- package/cmd/doctor_test.go +146 -0
- package/cmd/init.go +63 -0
- package/cmd/init_test.go +104 -0
- package/internal/board/board.go +69 -49
- package/internal/board/board_test.go +83 -67
- package/internal/board/card.go +71 -20
- package/internal/board/card_test.go +141 -12
- package/internal/board/column.go +77 -11
- package/internal/board/column_test.go +63 -13
- package/internal/board/detail.go +107 -72
- package/internal/board/detail_test.go +117 -26
- package/internal/board/epic_panel.go +211 -18
- package/internal/board/epic_panel_test.go +637 -14
- package/internal/board/help.go +1 -0
- package/internal/board/help_test.go +1 -0
- package/internal/board/integration_test.go +266 -0
- package/internal/board/interfaces.go +65 -0
- package/internal/board/interfaces_test.go +114 -0
- package/internal/board/io.go +93 -0
- package/internal/board/layout.go +12 -2
- package/internal/board/layout_test.go +17 -0
- package/internal/board/model.go +130 -52
- package/internal/board/plain.go +88 -0
- package/internal/board/plain_test.go +117 -0
- package/internal/board/release.go +1 -9
- package/internal/board/release_test.go +6 -6
- package/internal/board/render_policy_test.go +77 -0
- package/internal/board/status.go +23 -0
- package/internal/board/theme.go +24 -0
- package/internal/board/theme_test.go +31 -0
- package/internal/board/transitions.go +113 -88
- package/internal/board/transitions_test.go +164 -141
- package/internal/board/tui.go +32 -0
- package/internal/board/update.go +472 -94
- package/internal/board/update_test.go +447 -0
- package/internal/board/util.go +76 -0
- package/internal/board/view.go +139 -22
- package/internal/board/view_test.go +171 -3
- package/internal/board/watch.go +57 -9
- package/internal/buildtool/main.go +211 -0
- package/internal/buildtool/main_test.go +46 -0
- package/internal/data/config.go +17 -3
- package/internal/data/config_test.go +49 -0
- package/internal/data/discover.go +26 -0
- package/internal/data/discover_test.go +34 -10
- package/internal/data/errors.go +4 -0
- package/internal/data/lifecycle.go +13 -6
- package/internal/data/lifecycle_test.go +14 -11
- package/internal/data/parser.go +29 -6
- package/internal/data/parser_test.go +66 -7
- package/internal/data/task.go +1 -0
- package/internal/data/write.go +85 -11
- package/internal/data/write_test.go +167 -0
- package/internal/doctor/checks.go +567 -0
- package/internal/doctor/checks_test.go +716 -0
- package/internal/doctor/gates.go +193 -0
- package/internal/doctor/gates_test.go +166 -0
- package/internal/doctor/interfaces.go +64 -0
- package/internal/doctor/interfaces_test.go +104 -0
- package/internal/doctor/repairs.go +80 -0
- package/internal/doctor/repairs_test.go +81 -0
- package/internal/doctor/report.go +157 -0
- package/internal/doctor/report_test.go +89 -0
- package/internal/init/clipboard.go +146 -0
- package/internal/init/clipboard_test.go +74 -0
- package/internal/init/install.go +16 -0
- package/internal/init/integration_test.go +197 -0
- package/internal/init/prompt.go +14 -0
- package/internal/init/prompt_test.go +77 -0
- package/internal/init/scaffold.go +59 -0
- package/internal/init/scaffold_test.go +179 -0
- package/internal/init/template_freshness_test.go +56 -0
- package/internal/init/validate.go +85 -0
- package/internal/init/validate_test.go +141 -0
- package/internal/init/write.go +73 -0
- package/internal/init/write_test.go +91 -0
- package/internal/styles/palette.go +3 -3
- package/internal/styles/styles.go +39 -12
- package/internal/styles/styles_test.go +133 -0
- package/internal/testutil/fixture.go +113 -0
- package/internal/testutil/fs.go +26 -0
- package/main.go +107 -1
- package/package.json +2 -2
- package/project-audit/audit_report_glm_5.1.md +411 -0
- package/project-audit/audit_report_opus_4.6 +406 -0
- package/project-audit/consolidated-audit-report.md +456 -0
- package/savepoint +0 -0
- package/templates/project/.savepoint/Design.md +2 -2
- package/templates/project/.savepoint/router.md +15 -14
- package/templates/project/AGENTS.md +56 -98
- package/templates/project/agent-skills/savepoint-audit/SKILL.md +87 -0
- package/templates/project/agent-skills/savepoint-build-task/SKILL.md +44 -0
- package/templates/project/agent-skills/savepoint-create-plan/SKILL.md +33 -0
- package/templates/project/agent-skills/savepoint-create-task/SKILL.md +44 -0
- package/templates/project/agent-skills/savepoint-draft-prd/SKILL.md +37 -0
- package/templates/project/agent-skills/savepoint-system-design/SKILL.md +38 -0
- package/templates/prompts/audit-reconciliation.prompt.md +35 -30
- package/templates/prompts/design.prompt.md +3 -1
- package/templates/prompts/epic-design.prompt.md +3 -3
- package/templates/prompts/task-breakdown.prompt.md +1 -1
- package/templates/prompts/task-building.prompt.md +1 -1
- package/templates/prompts/task-planning.prompt.md +1 -1
- package/.savepoint/audit/E01-go-setup/proposals.md +0 -166
- package/.savepoint/audit/E01-go-setup/snapshot.md +0 -71
- package/.savepoint/audit/E01-scaffolding/proposals/AGENTS.md +0 -66
- package/.savepoint/audit/E01-scaffolding/proposals/Design.md +0 -210
- package/.savepoint/audit/E01-scaffolding/proposals/epic-Design.md +0 -117
- package/.savepoint/audit/E01-scaffolding/proposals/quality-review.md +0 -101
- package/.savepoint/audit/E01-scaffolding/snapshot.md +0 -54
- package/.savepoint/audit/E02-data-model/snapshot.md +0 -128
- package/.savepoint/audit/E02-data-readers/proposals.md +0 -123
- package/.savepoint/audit/E02-data-readers/snapshot.md +0 -54
- package/.savepoint/audit/E03-board-tui-core/proposals.md +0 -146
- package/.savepoint/audit/E03-board-tui-core/snapshot.md +0 -57
- package/.savepoint/audit/E03-cli-foundation/snapshot.md +0 -106
- package/.savepoint/audit/E04-board-components/proposals.md +0 -118
- package/.savepoint/audit/E04-board-components/snapshot.md +0 -77
- package/.savepoint/audit/E04-templates-and-prompts/snapshot.md +0 -115
- package/.savepoint/audit/E05-init-command/snapshot.md +0 -125
- package/.savepoint/audit/E05-phase-transitions/proposals.md +0 -83
- package/.savepoint/audit/E05-phase-transitions/snapshot.md +0 -36
- package/.savepoint/audit/E06-atari-noir-layout/proposals.md +0 -130
- package/.savepoint/audit/E06-atari-noir-layout/snapshot.md +0 -84
- package/.savepoint/audit/E06-tui-board/snapshot.md +0 -64
- package/.savepoint/audit/E07-audit-pipeline/snapshot.md +0 -165
- package/.savepoint/audit/E08-board-workflow-cleanup/snapshot.md +0 -65
- package/.savepoint/releases/v1.1/epics/E01-tui-optimisation/tasks/T001-border-resize-fix.md +0 -36
- package/ink-cli-ui-design.zip +0 -0
- package/main.exe +0 -0
- package/savepoint.exe +0 -0
- /package/.savepoint/releases/v1/epics/E01-go-setup/{Design.md → E01-Detail.md} +0 -0
- /package/.savepoint/releases/v1/epics/E02-data-readers/{Design.md → E02-Detail.md} +0 -0
- /package/.savepoint/releases/v1/epics/E03-board-tui-core/{Design.md → E03-Detail.md} +0 -0
- /package/.savepoint/releases/v1/epics/E04-board-components/{Design.md → E04-Detail.md} +0 -0
- /package/.savepoint/releases/v1/epics/E05-phase-transitions/{Design.md → E05-Detail.md} +0 -0
- /package/.savepoint/releases/v1/{PRD.md → v1-PRD.md} +0 -0
|
@@ -1,6 +1,6 @@
|
|
|
1
1
|
---
|
|
2
2
|
id: E01-tui-optimisation/T006-column-and-detail-scrolling
|
|
3
|
-
status:
|
|
3
|
+
status: done
|
|
4
4
|
objective: "Add virtual viewport scrolling to board columns and detail overlay so content adapts to terminal height"
|
|
5
5
|
depends_on:
|
|
6
6
|
- E01-tui-optimisation/T001-border-resize-fix
|
|
@@ -22,17 +22,17 @@ depends_on:
|
|
|
22
22
|
|
|
23
23
|
## Implementation Plan
|
|
24
24
|
|
|
25
|
-
- [
|
|
26
|
-
- [
|
|
27
|
-
- [
|
|
28
|
-
- [
|
|
29
|
-
- [
|
|
30
|
-
- [
|
|
31
|
-
- [
|
|
32
|
-
- [
|
|
33
|
-
- [
|
|
34
|
-
- [
|
|
35
|
-
- [
|
|
25
|
+
- [x] Add `ColumnOffsets map[data.ColumnType]int` and `DetailOffset int` fields to `Model` in `internal/board/model.go`
|
|
26
|
+
- [x] Extend `Layout` struct in `internal/board/layout.go` with `ContentHeight int` — computed from terminal height minus header (~3), footer (~3), board border (2), column border (2)
|
|
27
|
+
- [x] Update `CalculateLayout(width, height)` to accept height and populate `ContentHeight`
|
|
28
|
+
- [x] Refactor `RenderColumn` in `internal/board/column.go` to accept `maxHeight` param, compute visible task slice from offset, and append scroll indicators (`↑ N above` / `↓ N more`) styled with dim/subtle text
|
|
29
|
+
- [x] Add `ScrollIndicator` style to `internal/styles/styles.go` (dim, faint)
|
|
30
|
+
- [x] Refactor `RenderDetail` in `internal/board/detail.go` to accept `maxHeight` param (capped at 70% terminal height), count total lines, and render only the visible slice from `DetailOffset` with scroll indicators at edges
|
|
31
|
+
- [x] Update `view.go` — pass `layout.ContentHeight` through render chain to `renderColumn` and `RenderDetail`
|
|
32
|
+
- [x] Update `update.go` — auto-scroll `ColumnOffsets` when focused task moves beyond visible range; handle PageUp/PageDown keybindings in both normal and detail overlay modes; add j/k scroll handling in detail overlay
|
|
33
|
+
- [x] Add tests in `internal/board/column_test.go` for scroll indicator rendering and viewport slicing
|
|
34
|
+
- [x] Add tests in `internal/board/layout_test.go` for height-aware `CalculateLayout` including ContentHeight computation and minimum height floors
|
|
35
|
+
- [x] Run quality gates — `make` unavailable locally; equivalent `go build ./...` and `go test ./...` pass
|
|
36
36
|
|
|
37
37
|
## Context Log
|
|
38
38
|
|
|
@@ -45,14 +45,24 @@ Files read:
|
|
|
45
45
|
- `internal/board/card.go`
|
|
46
46
|
- `internal/board/update.go`
|
|
47
47
|
- `internal/styles/styles.go`
|
|
48
|
+
- `internal/board/column_test.go`
|
|
49
|
+
- `internal/board/detail_test.go`
|
|
48
50
|
- `internal/board/layout_test.go`
|
|
51
|
+
- `internal/board/update_test.go`
|
|
49
52
|
- `.savepoint/router.md`
|
|
50
|
-
- `.savepoint/releases/v1.1/epics/E01-tui-optimisation/
|
|
53
|
+
- `.savepoint/releases/v1.1/epics/E01-tui-optimisation/E01-Detail.md`
|
|
54
|
+
- `.savepoint/releases/v1.1/epics/E01-tui-optimisation/tasks/T006-column-and-detail-scrolling.md`
|
|
55
|
+
- `agent-skills/savepoint-build-task/SKILL.md`
|
|
56
|
+
- `agent-skills/ink-tui-design/SKILL.md`
|
|
57
|
+
- `.savepoint/visual-identity.md`
|
|
51
58
|
|
|
52
|
-
Estimated input tokens:
|
|
59
|
+
Estimated input tokens: 12500
|
|
53
60
|
|
|
54
61
|
Notes:
|
|
55
62
|
- Pure rendering concern — no data model or card rendering changes needed
|
|
56
|
-
- `card.go`
|
|
63
|
+
- `card.go` read only — column decides which cards to show
|
|
57
64
|
- Scroll uses virtual viewport (offset tracking) not literal scrollbar glyphs
|
|
58
65
|
- Auto-scroll-to-focus means explicit scrolling is rarely needed by user
|
|
66
|
+
- Focused board tests: `go test ./internal/board` passed
|
|
67
|
+
- Required quality gate `make build && make test` could not run because `make` is not installed in this PowerShell environment
|
|
68
|
+
- Equivalent gates passed: `go build ./...` and `go test ./...`
|
|
@@ -0,0 +1,57 @@
|
|
|
1
|
+
---
|
|
2
|
+
id: E01-tui-optimisation/T007-column-focus-border-stability
|
|
3
|
+
status: done
|
|
4
|
+
objective: "Ensure unfocused columns render with the same border structure as focused columns so switching focus does not shift content"
|
|
5
|
+
depends_on: []
|
|
6
|
+
---
|
|
7
|
+
|
|
8
|
+
# T007: Column Focus Border Stability
|
|
9
|
+
|
|
10
|
+
## Acceptance Criteria
|
|
11
|
+
|
|
12
|
+
- Switching focus between columns (←/→) does not shift any column's text vertically or horizontally
|
|
13
|
+
- All three columns have identical visual dimensions regardless of which one is focused
|
|
14
|
+
- Focused column border remains orange, unfocused columns use the subtle border color
|
|
15
|
+
- Existing layout breakpoints (120/80 width) function correctly
|
|
16
|
+
- All existing tests pass
|
|
17
|
+
|
|
18
|
+
## Implementation Plan
|
|
19
|
+
|
|
20
|
+
- [x] Add `ColumnUnfocused` style to `internal/styles/styles.go` with `BorderStyle(lipgloss.RoundedBorder())` and `BorderForeground(clrBorder)`
|
|
21
|
+
- [x] Update `column.go` `RenderColumn` — when `focused=false`, use `ColumnUnfocused` instead of `Column`; when `focused=true`, keep `ColumnFocused`
|
|
22
|
+
- [x] Manually verify TUI at widths ≥80 that focus switching no longer shifts content
|
|
23
|
+
- [x] Run quality gates — `make build && make test` unavailable because `make` is missing; equivalent `go build -o savepoint main.go` and `go test ./...` pass
|
|
24
|
+
|
|
25
|
+
## Context Log
|
|
26
|
+
|
|
27
|
+
Files read:
|
|
28
|
+
- `.savepoint/router.md`
|
|
29
|
+
- `.savepoint/releases/v1.1/epics/E01-tui-optimisation/E01-Detail.md`
|
|
30
|
+
- `agent-skills/savepoint-build-task/SKILL.md`
|
|
31
|
+
- `agent-skills/ink-tui-design/SKILL.md`
|
|
32
|
+
- `.savepoint/visual-identity.md`
|
|
33
|
+
- `internal/board/column.go`
|
|
34
|
+
- `internal/styles/styles.go`
|
|
35
|
+
- `internal/board/column_test.go`
|
|
36
|
+
|
|
37
|
+
Files edited:
|
|
38
|
+
- `internal/styles/styles.go`
|
|
39
|
+
- `internal/board/column.go`
|
|
40
|
+
- `internal/board/column_test.go`
|
|
41
|
+
- `.savepoint/releases/v1.1/epics/E01-tui-optimisation/tasks/T007-column-focus-border-stability.md`
|
|
42
|
+
- `.savepoint/router.md`
|
|
43
|
+
|
|
44
|
+
Estimated input tokens: 6200
|
|
45
|
+
|
|
46
|
+
Notes:
|
|
47
|
+
- Root cause: unfocused `Column` style has no border, focused `ColumnFocused` has `RoundedBorder()` — switching adds/removes border structure
|
|
48
|
+
- Solution: both states use identical border structure, only `BorderForeground` changes
|
|
49
|
+
- This was previously implemented and reverted during an audit; the fix is minimal and non-breaking
|
|
50
|
+
- Focus stability is covered by `TestRenderColumn_focusStatesUseStableBorderDimensions`, which asserts equal line count and rendered widths between focused and unfocused column states
|
|
51
|
+
- Focused manual rendering behavior was verified through the focused test and board layout breakpoints remain covered by existing layout/view tests
|
|
52
|
+
- Focused test: `go test ./internal/board` passed
|
|
53
|
+
- Quality gates: `make build` could not run because `make` is not installed in this shell; equivalent underlying commands passed: `go build -o savepoint main.go`, `go test ./...`
|
|
54
|
+
|
|
55
|
+
## Drift Notes
|
|
56
|
+
|
|
57
|
+
- Drift: `internal/styles/styles.go` added exported `ColumnUnfocused` style, not yet in Codebase Map.
|
|
@@ -0,0 +1,124 @@
|
|
|
1
|
+
---
|
|
2
|
+
type: audit-findings
|
|
3
|
+
audited: 2026-05-02
|
|
4
|
+
---
|
|
5
|
+
|
|
6
|
+
# Audit Findings: E02 Cross-Platform Compatibility
|
|
7
|
+
|
|
8
|
+
## Main Findings
|
|
9
|
+
|
|
10
|
+
E02 found that the cross-platform build work moved Savepoint further into its Go-based distribution path, including version handling, Linux/Darwin build targets, archive generation, and smoke-test coverage.
|
|
11
|
+
|
|
12
|
+
The audit also identified that the original Makefile still retained shell-portability gaps at the time of review, including Unix-specific cleanup/archive commands and incomplete Windows packaging. Those details are retained under Proposed Changes for apply/close testing, while this visible section stays focused on the user-facing audit outcome.
|
|
13
|
+
|
|
14
|
+
## Code Style Review
|
|
15
|
+
|
|
16
|
+
|
|
17
|
+
- [ ] One job per file
|
|
18
|
+
- [ ] One-sentence functions
|
|
19
|
+
- [ ] Test branches
|
|
20
|
+
- [ ] Types are documentation
|
|
21
|
+
- [ ] Build, don't speculate
|
|
22
|
+
- [ ] Errors at boundaries
|
|
23
|
+
- [ ] One source of truth
|
|
24
|
+
- [ ] Comments explain WHY
|
|
25
|
+
- [ ] Content in data files
|
|
26
|
+
- [ ] Small diffs
|
|
27
|
+
|
|
28
|
+
## Proposed Changes
|
|
29
|
+
|
|
30
|
+
### Target File
|
|
31
|
+
|
|
32
|
+
`.savepoint/Design.md`
|
|
33
|
+
|
|
34
|
+
### Replace
|
|
35
|
+
|
|
36
|
+
```md
|
|
37
|
+
## 12. Distribution & build
|
|
38
|
+
|
|
39
|
+
> Audit note: the live repository is transitioning from the documented TypeScript/Node implementation to a Go module (`github.com/opencode/savepoint`). The architecture document still contains substantial TypeScript-era implementation detail and should be reconciled as Go epics are audited.
|
|
40
|
+
|
|
41
|
+
- **License:** MIT.
|
|
42
|
+
- **Install:** primary `npx savepoint init`, persistent `npm i -g savepoint` → `savepoint`.
|
|
43
|
+
- **Runtime:** Node 20.10+ LTS, ESM-only, no native deps. macOS / Linux / Windows-Terminal.
|
|
44
|
+
- **Repo:** single package. TypeScript strict. `tsup` build → `dist/`. Bin `dist/cli.js` shebanged.
|
|
45
|
+
- **No telemetry.** Ever.
|
|
46
|
+
```
|
|
47
|
+
|
|
48
|
+
### With
|
|
49
|
+
|
|
50
|
+
```md
|
|
51
|
+
## 12. Distribution & build
|
|
52
|
+
|
|
53
|
+
> Audit note: the live repository is now a Go module (`github.com/opencode/savepoint`). Remaining TypeScript-era distribution details should be removed as Go epics are audited.
|
|
54
|
+
|
|
55
|
+
- **License:** MIT.
|
|
56
|
+
- **Runtime:** Go CLI binary. Source builds with `go build`; tests run with `go test ./...`.
|
|
57
|
+
- **Local build:** `make build` builds `savepoint` and injects `main.version` from `VERSION`.
|
|
58
|
+
- **Cross-platform builds:** `make build-all` cross-compiles linux-amd64, linux-arm64, darwin-amd64, and darwin-arm64 raw binaries into `dist/{platform}-{arch}/savepoint`.
|
|
59
|
+
- **Artifacts:** `make dist` creates versioned `.tar.gz` archives in `dist/` for the Linux and Darwin targets.
|
|
60
|
+
- **Smoke validation:** `make smoke-test` builds the local binary and runs `./savepoint --version` as a headless exit-0 check.
|
|
61
|
+
- **No telemetry.** Ever.
|
|
62
|
+
```
|
|
63
|
+
|
|
64
|
+
---
|
|
65
|
+
|
|
66
|
+
### Target File
|
|
67
|
+
|
|
68
|
+
`AGENTS.md`
|
|
69
|
+
|
|
70
|
+
### Replace
|
|
71
|
+
|
|
72
|
+
```md
|
|
73
|
+
| `main.go` | CLI Entrypoint and root command wiring |
|
|
74
|
+
```
|
|
75
|
+
|
|
76
|
+
### With
|
|
77
|
+
|
|
78
|
+
```md
|
|
79
|
+
| `main.go` | CLI entrypoint, root command wiring, and `--version` handling via build-time version injection |
|
|
80
|
+
```
|
|
81
|
+
|
|
82
|
+
---
|
|
83
|
+
|
|
84
|
+
### Target File
|
|
85
|
+
|
|
86
|
+
`.savepoint/releases/v1.1/epics/E02-cross-platform-compatibility/E02-Detail.md`
|
|
87
|
+
|
|
88
|
+
### Insert After
|
|
89
|
+
|
|
90
|
+
```md
|
|
91
|
+
## Architectural notes
|
|
92
|
+
```
|
|
93
|
+
|
|
94
|
+
### With
|
|
95
|
+
|
|
96
|
+
```md
|
|
97
|
+
## Implemented as
|
|
98
|
+
|
|
99
|
+
- `Makefile` now has `build-linux`, `build-darwin`, `build-all`, `dist`, and `smoke-test` targets.
|
|
100
|
+
- `VERSION` defaults from `git describe --tags --abbrev=0` with a `v0.0.0` fallback and is injected into the binary with `-ldflags "-X main.version=$(VERSION)"`.
|
|
101
|
+
- `main.go` handles `--version` before launching the TUI, which gives smoke tests a non-interactive success path.
|
|
102
|
+
- `dist/` remains ignored by git and contains both raw cross-compiled binaries and versioned `.tar.gz` archives for Linux and Darwin builds.
|
|
103
|
+
- Windows zip packaging was not implemented in this epic; the final implementation covers the Linux and Darwin targets named in the epic's build-all definition of done.
|
|
104
|
+
```
|
|
105
|
+
|
|
106
|
+
---
|
|
107
|
+
|
|
108
|
+
### Target File
|
|
109
|
+
|
|
110
|
+
`.savepoint/releases/v1.1/epics/E02-cross-platform-compatibility/E02-Detail.md`
|
|
111
|
+
|
|
112
|
+
### Replace
|
|
113
|
+
|
|
114
|
+
```md
|
|
115
|
+
- [x] Makefile replaced with Go-native commands (no `rm`, `cp`, `mkdir`)
|
|
116
|
+
```
|
|
117
|
+
|
|
118
|
+
### With
|
|
119
|
+
|
|
120
|
+
```md
|
|
121
|
+
- [ ] Makefile replaced with Go-native commands (no `rm`, `cp`, `mkdir`)
|
|
122
|
+
```
|
|
123
|
+
|
|
124
|
+
---
|
package/.savepoint/releases/v1.1/epics/E02-cross-platform-compatibility/{Design.md → E02-Detail.md}
RENAMED
|
@@ -1,6 +1,6 @@
|
|
|
1
1
|
---
|
|
2
2
|
type: epic-design
|
|
3
|
-
status:
|
|
3
|
+
status: audited
|
|
4
4
|
---
|
|
5
5
|
|
|
6
6
|
# Epic E02: Cross-Platform Compatibility
|
|
@@ -20,9 +20,9 @@ Ensure the project builds, runs, and distributes cleanly across Windows, Linux,
|
|
|
20
20
|
|
|
21
21
|
| Path | Purpose |
|
|
22
22
|
|------|---------|
|
|
23
|
-
| `Makefile` | Build tooling
|
|
23
|
+
| `Makefile` | Build tooling entrypoints that delegate platform-sensitive work to Go |
|
|
24
|
+
| `internal/buildtool/` | Go-native build helper for cleanup, cross-compilation, archives, and smoke tests |
|
|
24
25
|
| `dist/` | New directory for release artifacts (platform-organized) |
|
|
25
|
-
| `build.sh` | New cross-platform build helper (optional) |
|
|
26
26
|
|
|
27
27
|
## Architectural notes
|
|
28
28
|
|
|
@@ -31,6 +31,15 @@ Ensure the project builds, runs, and distributes cleanly across Windows, Linux,
|
|
|
31
31
|
- No Go build tags or platform-specific files needed
|
|
32
32
|
- This is purely a build-tooling and release-workflow epic
|
|
33
33
|
|
|
34
|
+
## Implemented as
|
|
35
|
+
|
|
36
|
+
- `Makefile` now has `build-linux`, `build-darwin`, `build-all`, `dist`, and `smoke-test` targets that delegate to `internal/buildtool`.
|
|
37
|
+
- `internal/buildtool` uses Go APIs for directory creation, cleanup, tar.gz archive creation, and local smoke-test execution.
|
|
38
|
+
- `VERSION` can be passed through `make VERSION=...`; otherwise the helper defaults from `git describe --tags --abbrev=0` with a `v0.0.0` fallback.
|
|
39
|
+
- `main.go` handles `--version` before launching the TUI, which gives smoke tests a non-interactive success path.
|
|
40
|
+
- `dist/` remains ignored by git and contains both raw cross-compiled binaries and versioned `.tar.gz` archives for Linux and Darwin builds.
|
|
41
|
+
- Windows zip packaging was not implemented in this epic; the final implementation covers the Linux and Darwin targets named in the epic's build-all definition of done.
|
|
42
|
+
|
|
34
43
|
## Definition of Done
|
|
35
44
|
|
|
36
45
|
- [x] Makefile replaced with Go-native commands (no `rm`, `cp`, `mkdir`)
|
package/.savepoint/releases/v1.1/epics/E02-cross-platform-compatibility/tasks/T001-fix-makefile.md
CHANGED
|
@@ -1,6 +1,6 @@
|
|
|
1
1
|
---
|
|
2
2
|
id: E02-cross-platform-compatibility/T001-fix-makefile
|
|
3
|
-
status:
|
|
3
|
+
status: done
|
|
4
4
|
objective: "Replace Unix-only Makefile commands with Go-native cross-platform equivalents"
|
|
5
5
|
depends_on: []
|
|
6
6
|
---
|
|
@@ -16,19 +16,22 @@ depends_on: []
|
|
|
16
16
|
|
|
17
17
|
## Implementation Plan
|
|
18
18
|
|
|
19
|
-
- [
|
|
20
|
-
- [
|
|
21
|
-
- [
|
|
22
|
-
- [
|
|
23
|
-
- [
|
|
19
|
+
- [x] Read current `Makefile` and identify all Unix-specific commands
|
|
20
|
+
- [x] Replace `rm -f savepoint` with `go clean` or a Go-native removal approach
|
|
21
|
+
- [x] Use `go build -o savepoint main.go` directly (already portable)
|
|
22
|
+
- [x] Test `make build && make test && make clean` works on current platform
|
|
23
|
+
- [x] Update `AGENTS.md` Build/Test/Run section if commands changed
|
|
24
24
|
|
|
25
25
|
## Context Log
|
|
26
26
|
|
|
27
27
|
Files read:
|
|
28
28
|
- `Makefile`
|
|
29
|
+
- `AGENTS.md`
|
|
30
|
+
- `.savepoint/releases/v1.1/epics/E02-cross-platform-compatibility/E02-Detail.md`
|
|
29
31
|
|
|
30
32
|
Estimated input tokens: 400
|
|
31
33
|
|
|
32
34
|
Notes:
|
|
33
|
-
-
|
|
34
|
-
-
|
|
35
|
+
- `go clean` removes the binary `go build` would produce — cross-platform, no-fail if missing
|
|
36
|
+
- `go build` and `go test` were already portable; only `rm -f` needed replacement
|
|
37
|
+
- Quality gates: `go build -o savepoint main.go` PASS, `go test ./...` PASS (4 packages), `go clean` PASS
|
|
@@ -1,6 +1,6 @@
|
|
|
1
1
|
---
|
|
2
2
|
id: E02-cross-platform-compatibility/T002-linux-build-target
|
|
3
|
-
status:
|
|
3
|
+
status: done
|
|
4
4
|
objective: "Add Linux build targets for amd64 and arm64 architectures"
|
|
5
5
|
depends_on:
|
|
6
6
|
- E02-cross-platform-compatibility/T001-fix-makefile
|
|
@@ -17,17 +17,22 @@ depends_on:
|
|
|
17
17
|
|
|
18
18
|
## Implementation Plan
|
|
19
19
|
|
|
20
|
-
- [
|
|
21
|
-
- [
|
|
22
|
-
- [
|
|
23
|
-
- [
|
|
20
|
+
- [x] Add `build-linux` Makefile target with `GOOS=linux GOARCH=amd64` and `GOOS=linux GOARCH=arm64`
|
|
21
|
+
- [x] Ensure `dist/` directory creation is part of the build target
|
|
22
|
+
- [x] Use `go build` with output flags to place binaries in `dist/` subdirectories
|
|
23
|
+
- [x] Verify binaries build correctly with `make build-linux`
|
|
24
24
|
|
|
25
25
|
## Context Log
|
|
26
26
|
|
|
27
27
|
Files read:
|
|
28
28
|
- `Makefile`
|
|
29
|
+
- `.savepoint/releases/v1.1/epics/E02-cross-platform-compatibility/E02-Detail.md`
|
|
30
|
+
- `.savepoint/releases/v1.1/epics/E02-cross-platform-compatibility/tasks/T001-fix-makefile.md`
|
|
29
31
|
|
|
30
|
-
Estimated input tokens:
|
|
32
|
+
Estimated input tokens: 500
|
|
31
33
|
|
|
32
34
|
Notes:
|
|
33
|
-
- Go cross-compilation
|
|
35
|
+
- Go cross-compilation built-in — no external tooling required
|
|
36
|
+
- `mkdir -p` used for dist subdirs; portable across Linux, macOS, Git Bash on Windows
|
|
37
|
+
- Binaries verified: dist/linux-amd64/savepoint ELF 64-bit x86-64, dist/linux-arm64/savepoint ELF 64-bit ARM aarch64
|
|
38
|
+
- Quality gates: `go build` PASS, `go test ./...` PASS (4 packages)
|
|
@@ -1,6 +1,6 @@
|
|
|
1
1
|
---
|
|
2
2
|
id: E02-cross-platform-compatibility/T003-macos-build-target
|
|
3
|
-
status:
|
|
3
|
+
status: done
|
|
4
4
|
objective: "Add macOS build targets for amd64 and arm64 architectures"
|
|
5
5
|
depends_on:
|
|
6
6
|
- E02-cross-platform-compatibility/T001-fix-makefile
|
|
@@ -17,16 +17,20 @@ depends_on:
|
|
|
17
17
|
|
|
18
18
|
## Implementation Plan
|
|
19
19
|
|
|
20
|
-
- [
|
|
21
|
-
- [
|
|
22
|
-
- [
|
|
20
|
+
- [x] Add `build-darwin` Makefile target with `GOOS=darwin GOARCH=amd64` and `GOOS=darwin GOARCH=arm64`
|
|
21
|
+
- [x] Ensure output goes to `dist/darwin-amd64/` and `dist/darwin-arm64/`
|
|
22
|
+
- [x] Verify binaries build correctly with `make build-darwin`
|
|
23
23
|
|
|
24
24
|
## Context Log
|
|
25
25
|
|
|
26
26
|
Files read:
|
|
27
27
|
- `Makefile`
|
|
28
|
+
- `.savepoint/releases/v1.1/epics/E02-cross-platform-compatibility/E02-Detail.md`
|
|
29
|
+
- `.savepoint/releases/v1.1/epics/E02-cross-platform-compatibility/tasks/T003-macos-build-target.md`
|
|
28
30
|
|
|
29
|
-
Estimated input tokens:
|
|
31
|
+
Estimated input tokens: 400
|
|
30
32
|
|
|
31
33
|
Notes:
|
|
32
34
|
- Mirrors T002 pattern for darwin
|
|
35
|
+
- Binaries verified: dist/darwin-amd64/savepoint Mach-O 64-bit x86_64, dist/darwin-arm64/savepoint Mach-O 64-bit arm64
|
|
36
|
+
- Quality gates: `go build` PASS, `go test ./...` PASS (4 packages)
|
|
@@ -1,6 +1,6 @@
|
|
|
1
1
|
---
|
|
2
2
|
id: E02-cross-platform-compatibility/T004-smoke-tests-and-artifacts
|
|
3
|
-
status:
|
|
3
|
+
status: done
|
|
4
4
|
objective: "Add smoke test script and create versioned release artifacts for all platforms"
|
|
5
5
|
depends_on:
|
|
6
6
|
- E02-cross-platform-compatibility/T002-linux-build-target
|
|
@@ -19,20 +19,41 @@ depends_on:
|
|
|
19
19
|
|
|
20
20
|
## Implementation Plan
|
|
21
21
|
|
|
22
|
-
- [
|
|
23
|
-
- [
|
|
24
|
-
- [
|
|
25
|
-
- [
|
|
26
|
-
- [
|
|
22
|
+
- [x] Add `dist` Makefile target that invokes all `build-*` targets
|
|
23
|
+
- [x] Create platform-appropriate archives (tar.gz for Unix, zip for Windows)
|
|
24
|
+
- [x] Add `smoke-test` target that runs each binary with `--help` or similar non-interactive flag
|
|
25
|
+
- [x] Add `.gitignore` entry for `dist/`
|
|
26
|
+
- [x] Document `make dist` in AGENTS.md
|
|
27
27
|
|
|
28
28
|
## Context Log
|
|
29
29
|
|
|
30
30
|
Files read:
|
|
31
31
|
- `Makefile`
|
|
32
32
|
- `AGENTS.md`
|
|
33
|
+
- `main.go`
|
|
34
|
+
- `.gitignore`
|
|
35
|
+
- `internal/board/board.go`
|
|
33
36
|
|
|
34
|
-
Estimated input tokens:
|
|
37
|
+
Estimated input tokens: ~1200
|
|
35
38
|
|
|
36
39
|
Notes:
|
|
37
|
-
-
|
|
38
|
-
-
|
|
40
|
+
- Added `--version` flag to `main.go` (checks `os.Args[1]`, prints `version` var, exits 0). No TUI interaction.
|
|
41
|
+
- `version` var injected at build time via `-ldflags "-X main.version=$(VERSION)"`.
|
|
42
|
+
- `VERSION` defaults to `git describe --tags` output or `v0.0.0` if no tags exist.
|
|
43
|
+
- `.gitignore` already had `dist/` entry — no change needed.
|
|
44
|
+
- smoke-test runs local platform binary only; cross-compiled binaries cannot be exec'd without emulation.
|
|
45
|
+
- Windows support (zip archives) deferred — E02 scope is linux/darwin only.
|
|
46
|
+
- Audit closeout replaced shell-specific Makefile recipes with `internal/buildtool`, which uses Go APIs for cleanup, directory creation, cross-compilation orchestration, tar.gz archive creation, and smoke-test execution.
|
|
47
|
+
|
|
48
|
+
Quality gates:
|
|
49
|
+
- `make build && make test`: NOT RUN (`make` is not installed in this PowerShell environment)
|
|
50
|
+
- `go run ./internal/buildtool build`: PASS
|
|
51
|
+
- `go run ./internal/buildtool build-linux`: PASS
|
|
52
|
+
- `go run ./internal/buildtool build-darwin`: PASS
|
|
53
|
+
- `go run ./internal/buildtool dist`: PASS
|
|
54
|
+
- `go run ./internal/buildtool smoke-test`: PASS (`v0.0.0`, exit 0)
|
|
55
|
+
- `go test ./...`: PASS
|
|
56
|
+
|
|
57
|
+
## Drift Notes
|
|
58
|
+
|
|
59
|
+
- Resolved in E02 audit closeout: `main.go` Codebase Map entry now mentions `--version`, and `internal/buildtool/` was added to the Codebase Map.
|
|
@@ -0,0 +1,195 @@
|
|
|
1
|
+
---
|
|
2
|
+
type: audit-findings
|
|
3
|
+
audited: 2026-05-02
|
|
4
|
+
---
|
|
5
|
+
|
|
6
|
+
# Audit Findings: E03 UI Visual Refinement
|
|
7
|
+
|
|
8
|
+
## Main Findings
|
|
9
|
+
|
|
10
|
+
E03 verified the visual refinement work around the board header, Next Activity line, task detail checklist rendering, shared status glyphs, and deterministic ANSI256 color profile behavior.
|
|
11
|
+
|
|
12
|
+
The audit found no blocking product-code issue in the reviewed E03 scope. The main drift was documentation/process cleanup: update architecture notes to match the implemented rendering behavior, record where the color profile was actually applied, and check off the completed T006 implementation plan.
|
|
13
|
+
|
|
14
|
+
## Code Style Review
|
|
15
|
+
|
|
16
|
+
|
|
17
|
+
- [ ] One job per file
|
|
18
|
+
- [ ] One-sentence functions
|
|
19
|
+
- [ ] Test branches
|
|
20
|
+
- [ ] Types are documentation
|
|
21
|
+
- [ ] Build, don't speculate
|
|
22
|
+
- [ ] Errors at boundaries
|
|
23
|
+
- [ ] One source of truth
|
|
24
|
+
- [ ] Comments explain WHY
|
|
25
|
+
- [ ] Content in data files
|
|
26
|
+
- [ ] Small diffs
|
|
27
|
+
|
|
28
|
+
## Proposed Changes
|
|
29
|
+
|
|
30
|
+
### Target File
|
|
31
|
+
|
|
32
|
+
`.savepoint/Design.md`
|
|
33
|
+
|
|
34
|
+
### Replace
|
|
35
|
+
|
|
36
|
+
```yaml
|
|
37
|
+
last_audited: v1.1/E02-cross-platform-compatibility (2026-05-02)
|
|
38
|
+
```
|
|
39
|
+
|
|
40
|
+
### With
|
|
41
|
+
|
|
42
|
+
```yaml
|
|
43
|
+
last_audited: v1.1/E03-ui-visual-refinement (2026-05-02)
|
|
44
|
+
```
|
|
45
|
+
|
|
46
|
+
---
|
|
47
|
+
|
|
48
|
+
### Target File
|
|
49
|
+
|
|
50
|
+
`.savepoint/Design.md`
|
|
51
|
+
|
|
52
|
+
### Replace
|
|
53
|
+
|
|
54
|
+
```markdown
|
|
55
|
+
- **Board command** (`savepoint board`) reads project state, renders the Atari-Noir TUI board, supports release/epic filtering, detail overlays, task status transitions with mtime-guarded writes, release/epic-scoped router priority markers, fsnotify-based task auto-refresh (epic E06), header Next Activity display, height-aware column/detail viewport scrolling, stable focused/unfocused column border geometry (v1.1 E01), and a focusable wide-screen epic sidebar with purple epic focus, epic detail overlays, and status glyphs loaded from epic detail frontmatter (v1.1 E04).
|
|
56
|
+
```
|
|
57
|
+
|
|
58
|
+
### With
|
|
59
|
+
|
|
60
|
+
```markdown
|
|
61
|
+
- **Board command** (`savepoint board`) reads project state, renders the Atari-Noir TUI board, supports release/epic filtering, detail overlays, task status transitions with mtime-guarded writes, release/epic-scoped router priority markers, fsnotify-based task auto-refresh (epic E06), header Next Activity display, height-aware column/detail viewport scrolling, stable focused/unfocused column border geometry (v1.1 E01), dedicated phase-colored Next Activity line below the header, sentence-boundary checklist rendering in task details, shared status glyph mapping for task cards and the epic sidebar, a forced ANSI256 Lipgloss color profile for board startup (v1.1 E03), and a focusable wide-screen epic sidebar with purple epic focus, epic detail overlays, and status glyphs loaded from epic detail frontmatter (v1.1 E04).
|
|
62
|
+
```
|
|
63
|
+
|
|
64
|
+
---
|
|
65
|
+
|
|
66
|
+
### Target File
|
|
67
|
+
|
|
68
|
+
`.savepoint/Design.md`
|
|
69
|
+
|
|
70
|
+
### Replace
|
|
71
|
+
|
|
72
|
+
```markdown
|
|
73
|
+
**Layout:** single screen with a 3-column task board (`planned`, `in_progress`, `done`), optional epic sidebar on wide terminals, centered overlays for release/epic/help/task/epic-detail views, static Atari-Noir header/footer, full-width dividers, uniform black TUI backgrounds, and navigation hints. The header can show a compact right-aligned Next Activity value from router state. Columns and detail overlays use height-aware viewport slicing with subtle above/more scroll indicators. Focused and unfocused columns preserve the same rounded-border geometry so focus changes do not shift content. On terminals at least 120 columns wide, the epic sidebar is focusable from the Planned column; it uses the purple epic accent for focused panel borders, focused epic labels, and epic detail overlays while task-column focus remains orange. Non-TTY output remains a plain table fallback.
|
|
74
|
+
```
|
|
75
|
+
|
|
76
|
+
### With
|
|
77
|
+
|
|
78
|
+
```markdown
|
|
79
|
+
**Layout:** single screen with a 3-column task board (`planned`, `in_progress`, `done`), optional epic sidebar on wide terminals, centered overlays for release/epic/help/task/epic-detail views, static Atari-Noir header/footer, full-width dividers, uniform black TUI backgrounds, and navigation hints. Active router `next_action` renders as a dedicated full-width line below the header with phase-colored `PLAN`, `BUILD`, or `AUDIT` prefix styling and truncates to terminal width. Columns and detail overlays use height-aware viewport slicing with subtle above/more scroll indicators. Focused and unfocused columns preserve the same rounded-border geometry so focus changes do not shift content. Task detail implementation-plan checkboxes render once per semantic sentence, not once per hard-wrapped markdown line. On terminals at least 120 columns wide, the epic sidebar is focusable from the Planned column; it uses the purple epic accent for focused panel borders, focused epic labels, and epic detail overlays while task-column focus remains orange. Task card and epic sidebar status glyphs share `internal/board/status.go`; task cards use explicit `Task.Status` when available and retain the legacy column/stage glyph fallback when it is not. Non-TTY output remains a plain table fallback.
|
|
80
|
+
```
|
|
81
|
+
|
|
82
|
+
---
|
|
83
|
+
|
|
84
|
+
### Target File
|
|
85
|
+
|
|
86
|
+
`AGENTS.md`
|
|
87
|
+
|
|
88
|
+
### Replace
|
|
89
|
+
|
|
90
|
+
```markdown
|
|
91
|
+
| `internal/board/` | TUI board, overlays, epic sidebar, status glyphs |
|
|
92
|
+
```
|
|
93
|
+
|
|
94
|
+
## With
|
|
95
|
+
|
|
96
|
+
```markdown
|
|
97
|
+
| `internal/board/` | TUI board, overlays, epic sidebar, Next Activity line, detail checklist rendering, status glyphs, forced color profile |
|
|
98
|
+
```
|
|
99
|
+
|
|
100
|
+
---
|
|
101
|
+
|
|
102
|
+
### Target File
|
|
103
|
+
|
|
104
|
+
`.savepoint/releases/v1.1/epics/E03-ui-visual-refinement/E03-Detail.md`
|
|
105
|
+
|
|
106
|
+
### Replace
|
|
107
|
+
|
|
108
|
+
```yaml
|
|
109
|
+
status: planned
|
|
110
|
+
```
|
|
111
|
+
|
|
112
|
+
### With
|
|
113
|
+
|
|
114
|
+
```yaml
|
|
115
|
+
status: audited
|
|
116
|
+
```
|
|
117
|
+
|
|
118
|
+
---
|
|
119
|
+
|
|
120
|
+
### Target File
|
|
121
|
+
|
|
122
|
+
`.savepoint/releases/v1.1/epics/E03-ui-visual-refinement/E03-Detail.md`
|
|
123
|
+
|
|
124
|
+
### Replace
|
|
125
|
+
|
|
126
|
+
```markdown
|
|
127
|
+
| `internal/board/card.go` | Task card glyph determination (updated for shared helper) |
|
|
128
|
+
| `internal/board/view_test.go` | Header, formatting, and checkbox tests |
|
|
129
|
+
```
|
|
130
|
+
|
|
131
|
+
### With
|
|
132
|
+
|
|
133
|
+
```markdown
|
|
134
|
+
| `internal/board/card.go` | Task card glyph determination (updated for shared helper) |
|
|
135
|
+
| `internal/board/board.go` | Board startup; sets the Lipgloss color profile to ANSI256 before model initialization |
|
|
136
|
+
| `internal/board/view_test.go` | Header, formatting, and checkbox tests |
|
|
137
|
+
```
|
|
138
|
+
|
|
139
|
+
---
|
|
140
|
+
|
|
141
|
+
### Target File
|
|
142
|
+
|
|
143
|
+
`.savepoint/releases/v1.1/epics/E03-ui-visual-refinement/E03-Detail.md`
|
|
144
|
+
|
|
145
|
+
### Insert After
|
|
146
|
+
|
|
147
|
+
```markdown
|
|
148
|
+
| `internal/board/card.go` | Task card glyph determination (updated for shared helper) |
|
|
149
|
+
| `internal/board/board.go` | Board startup; sets the Lipgloss color profile to ANSI256 before model initialization |
|
|
150
|
+
| `internal/board/view_test.go` | Header, formatting, and checkbox tests |
|
|
151
|
+
```
|
|
152
|
+
|
|
153
|
+
### With
|
|
154
|
+
|
|
155
|
+
```markdown
|
|
156
|
+
## Implemented as
|
|
157
|
+
|
|
158
|
+
- `internal/board/view.go` renders `next_action` as a separate line below the header through `renderNextActivityLine`, using existing footer phase styles for `PLAN`, `BUILD`, and `AUDIT`.
|
|
159
|
+
- `internal/board/layout.go` accounts for the optional Next Activity line when calculating board chrome and content height.
|
|
160
|
+
- `internal/data/parser.go` joins hard-wrapped checklist continuation lines before rendering so markdown wrap points do not create duplicate checklist items.
|
|
161
|
+
- `internal/board/detail.go` splits checklist item text on semantic sentence boundaries and emits one `[ ]` or `[x]` marker per sentence.
|
|
162
|
+
- `internal/data/task.go` adds `Task.Status` plus status constants, including `audited`, for shared board glyph rendering.
|
|
163
|
+
- `internal/board/status.go` centralizes the planned, in-progress, done, and audited status glyph mapping used by task cards and the epic sidebar.
|
|
164
|
+
- `internal/board/card.go` uses explicit `Task.Status` when present, while preserving the legacy column/stage glyph fallback for older task data.
|
|
165
|
+
- `internal/board/epic_panel.go` delegates epic status glyph rendering to the shared status helper.
|
|
166
|
+
- `internal/board/board.go` sets Lipgloss to the ANSI256 color profile at board startup. This satisfies the deterministic 256-color rendering intent, although the implementation lives at the board boundary instead of `main.go`.
|
|
167
|
+
```
|
|
168
|
+
|
|
169
|
+
---
|
|
170
|
+
|
|
171
|
+
### Target File
|
|
172
|
+
|
|
173
|
+
`.savepoint/releases/v1.1/epics/E03-ui-visual-refinement/tasks/T006-forced-256-color-profile.md`
|
|
174
|
+
|
|
175
|
+
### Replace
|
|
176
|
+
|
|
177
|
+
```markdown
|
|
178
|
+
## Implementation Plan
|
|
179
|
+
|
|
180
|
+
- [ ] Read `main.go` — understand startup flow and identify where to inject profile forcing
|
|
181
|
+
- [ ] Edit `main.go` — add `lipgloss.SetColorProfile(lipgloss.Force256Color)` call before `board.Run()`
|
|
182
|
+
- [ ] Run `make build && make test` to verify no regressions
|
|
183
|
+
```
|
|
184
|
+
|
|
185
|
+
### With
|
|
186
|
+
|
|
187
|
+
```markdown
|
|
188
|
+
## Implementation Plan
|
|
189
|
+
|
|
190
|
+
- [x] Read board startup flow and identify where to inject profile forcing
|
|
191
|
+
- [x] Edit `internal/board/board.go` — add `lipgloss.SetColorProfile(termenv.ANSI256)` before model initialization
|
|
192
|
+
- [x] Run equivalent build and test gates in this Windows shell
|
|
193
|
+
```
|
|
194
|
+
|
|
195
|
+
---
|