Skip to content

main -> release#387

Merged
saadqbal merged 2 commits into
mainfrom
develop
Jul 22, 2026
Merged

main -> release#387
saadqbal merged 2 commits into
mainfrom
develop

Conversation

@saadqbal

@saadqbal saadqbal commented Jul 22, 2026

Copy link
Copy Markdown
Collaborator

Summary

Related

Type of change

  • Feature
  • Bug fix
  • Tech-debt / refactor
  • Docs
  • Security / hardening

Test plan

Checklist

  • Tests added / updated and passing locally
  • go build ./..., go vet, and the Lint job's checks pass locally
  • Terminal output follows STYLE.md — Printer tones (no hardcoded colour/emoji), "secure environment" not "workspace"; bash scripts/check-style.sh passes
  • No secrets / credentials in the diff

Note

Medium Risk
Large user-facing copy and ingest questionnaire behavior change, mitigated by extensive golden tests and updated interactive tests; no auth or data-path logic changes.

Overview
Redesigns the interactive tb data ingest flow: four numbered PromptStep headers (intent → name → path → task), task-specific follow-ups as Section headers, and bare survey prompts so the ? line shows only the answer. The task picker lists task IDs with aligned blurbs instead of display names. Intro copy is shortened; ReviewShown suppresses duplicate "Ingest settings" in printLocalSummary after a guided review.

Adds TestCopyCatalog and testdata/golden/*.golden — per-command screen snapshots plus zz-all-strings.golden (AST-harvested copy backstop) so user-facing strings drift in CI.

STYLE.md documents guided-prompt spacing and welcomes intentional emoji; check-style.sh drops the emoji ban. submit.RenderSummary omits zero shortfall fields on clean runs. Printer.PromptStep is new; table-name validation errors are friendlier.

Reviewed by Cursor Bugbot for commit a482731. Bugbot is set up for automated code reviews on this repo. Configure here.

@saadqbal
saadqbal merged commit aebdb1d into main Jul 22, 2026
44 of 45 checks passed
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.

3 participants