Skip to content

Add agenttrace to terminal tools - #2

Open
luoyuctl wants to merge 2 commits into
yayxs:mainfrom
luoyuctl:add-agenttrace-terminal-tool
Open

Add agenttrace to terminal tools#2
luoyuctl wants to merge 2 commits into
yayxs:mainfrom
luoyuctl:add-agenttrace-terminal-tool

Conversation

@luoyuctl

Copy link
Copy Markdown

Adds agenttrace as a terminal AI coding tool in the English/Chinese README, MDX content, and the data source used by the site.

agenttrace is a local TUI for reviewing AI coding agent session history across cost, tokens, latency, tool failures, anomalies, health, and reports.

Checks:

  • pnpm install --frozen-lockfile
  • pnpm build
  • git diff --check

Note: pnpm build completed successfully. It emitted existing project warnings about ESLint option compatibility and package module type, but the Next.js production build and static generation succeeded.

@vercel

vercel Bot commented May 10, 2026

Copy link
Copy Markdown

Someone is attempting to deploy a commit to the yayxs' projects Team on Vercel.

A member of the Team first needs to authorize it.

@netlify

netlify Bot commented May 10, 2026

Copy link
Copy Markdown

Deploy Preview for ai-coding failed.

Name Link
🔨 Latest commit 9b6725c
🔍 Latest deploy log https://app.netlify.com/projects/ai-coding/deploys/6a00950222aa34000820aed9

@gemini-code-assist gemini-code-assist Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Code Review

This pull request adds 'agenttrace', a local TUI for monitoring AI coding agent sessions, to the project's documentation and tool list across several files, including English and Chinese versions of the README and MDX files. The reviewer suggested updating the tool name to Title Case in the data configuration for consistency with other entries.

Comment thread data/aiTools.ts Outdated
outputType: 'Terminal',
},
{
whatsTheName: 'agenttrace',

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

medium

For consistency with other tools in this list (e.g., Cursor, Windsurf, Aider, Warp), consider using Title Case for the tool name.

Suggested change
whatsTheName: 'agenttrace',
whatsTheName: 'Agenttrace',

@luoyuctl

Copy link
Copy Markdown
Author

Checked the open review note: the requested Title Case change in data/aiTools.ts is already addressed by the second commit (whatsTheName: "Agenttrace").

The remaining preview noise appears to be Vercel/Netlify deployment authorization for first-time fork builds, not a code change needed from this PR branch.

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.

1 participant