Skip to content

feat(#1249): show version on banner line - #1493

Merged
ralphbean merged 1 commit into
mainfrom
agent/1249-banner-version
May 26, 2026
Merged

feat(#1249): show version on banner line#1493
ralphbean merged 1 commit into
mainfrom
agent/1249-banner-version

Conversation

@fullsend-ai-coder

Copy link
Copy Markdown
Contributor

Add a version parameter to Printer.Banner() so the CLI version appears next to the brand name in terminal output. All call sites in admin.go and run.go now pass cli.Version(). This makes the version immediately visible in install logs and bug reports without requiring a separate --version call.

Note: pre-commit could not run due to a sandbox Go toolchain permission issue (not related to this change). Unit tests and go vet passed.


Closes #1249

Post-script verification

  • Branch is not main/master (agent/1249-banner-version)
  • Secret scan passed (gitleaks — addf9a24b1a9d937a2e2027b8c67a84f8d087739..HEAD)
  • Pre-commit hooks passed (authoritative run on runner)
  • Tests ran inside sandbox

Add a version parameter to Printer.Banner() so the CLI version
appears next to the brand name in terminal output. All call sites
in admin.go and run.go now pass cli.Version(). This makes the
version immediately visible in install logs and bug reports
without requiring a separate --version call.

Note: pre-commit could not run due to a sandbox Go toolchain
permission issue (not related to this change). Unit tests and
go vet passed.

Closes #1249

Signed-off-by: fullsend-code <fullsend-code@users.noreply.github.com>
@github-actions

Copy link
Copy Markdown

Site preview

Preview: https://3c9855fb-site.fullsend-ai.workers.dev

Commit: 03c22b1a3e60fad126a59f3e26d399620ae35cdc

@fullsend-ai-review

Copy link
Copy Markdown

Review

Findings

No findings.

Clean, well-scoped change. All 7 Banner() call sites updated consistently. Tests cover both versioned (v1.2.3) and dev (dev) cases. The Version() function defaults to "dev" via the package-level version var, so the banner always has meaningful content.

@fullsend-ai-review fullsend-ai-review Bot added the ready-for-merge All reviewers approved — ready to merge label May 26, 2026
@ralphbean
ralphbean added this pull request to the merge queue May 26, 2026
Merged via the queue into main with commit f6c8d35 May 26, 2026
11 of 12 checks passed
@ralphbean
ralphbean deleted the agent/1249-banner-version branch May 26, 2026 18:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ready-for-merge All reviewers approved — ready to merge

Projects

None yet

Development

Successfully merging this pull request may close these issues.

admin install: show version on banner line

1 participant