Skip to content

fix: use agent-neutral wording in explain empty-state message#1086

Merged
computermode merged 1 commit intoentireio:mainfrom
KuaaMU:fix/agent-neutral-message
May 1, 2026
Merged

fix: use agent-neutral wording in explain empty-state message#1086
computermode merged 1 commit intoentireio:mainfrom
KuaaMU:fix/agent-neutral-message

Conversation

@KuaaMU
Copy link
Copy Markdown
Contributor

@KuaaMU KuaaMU commented Apr 30, 2026

Problem

The entire explain command's empty-state message (when no checkpoints exist) hardcodes "a Claude session" — so it always references Claude even when a different agent (Copilot CLI, Gemini CLI, etc.) is configured.

Fix

Changed the message from "a Claude session" to "an agent session" in:

  1. cmd/entire/cli/explain.go:2286 — source message
  2. cmd/entire/cli/explain_test.go:1082 — test assertion

2 files, 2 lines changed.

Fixes #1079

Copilot AI review requested due to automatic review settings April 30, 2026 15:55
@KuaaMU KuaaMU requested a review from a team as a code owner April 30, 2026 15:55
Copy link
Copy Markdown

@claude claude Bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Claude Code Review

This pull request is from a fork — automated review is disabled. A repository maintainer can comment @claude review to run a one-time review.

Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Updates entire explain’s empty-state output to avoid hardcoding a specific agent name (Claude) when no checkpoints exist, aligning messaging with Entire CLI’s agent-agnostic design.

Changes:

  • Replaced “a Claude session” with “an agent session” in the entire explain no-checkpoints message.
  • Updated the corresponding unit test assertion to match the new wording.

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated no comments.

File Description
cmd/entire/cli/explain.go Makes the no-checkpoints help text agent-neutral (“an agent session”).
cmd/entire/cli/explain_test.go Updates the test to assert the agent-neutral wording.

Changed 'a Claude session' to 'an agent session' in explain.go and
explain_test.go so the message is accurate regardless of which agent
(Copilot CLI, Gemini CLI, etc.) is configured.

Fixes entireio#1079
@KuaaMU KuaaMU force-pushed the fix/agent-neutral-message branch from 11bcd99 to fc0a25c Compare May 1, 2026 03:47
@computermode
Copy link
Copy Markdown
Contributor

Thank you! Merging this in 🚀

@computermode computermode merged commit d8f8ad6 into entireio:main May 1, 2026
9 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

Bug: entire explain empty-state message always references Claude

3 participants