Skip to content

Sync agenteye-cli skill (agenteye@1e6ff16)#2

Open
NiveditJain wants to merge 1 commit into
mainfrom
sync/agenteye-cli
Open

Sync agenteye-cli skill (agenteye@1e6ff16)#2
NiveditJain wants to merge 1 commit into
mainfrom
sync/agenteye-cli

Conversation

@NiveditJain

@NiveditJain NiveditJain commented Jul 15, 2026

Copy link
Copy Markdown
Member

Automated mirror — do not hand-edit this folder.

Source of truth FailproofAI/agenteyecli/skill/
Target skills/agenteye-cli/
Triggered from agenteye main @ 1e6ff16

The skills repo's validate-skills.py passed. To change the skill, edit cli/skill/ in agenteye and re-run the Sync agenteye-cli skill workflow — this same PR refreshes.

Summary by CodeRabbit

  • Documentation
    • Clarified when to use the AgentEye CLI and distinguished telemetry inspection from deployment operations.
    • Documented that event listings and error results omit payloads by default.
    • Added guidance for retrieving raw payloads with --full or --fields payload, restricted to a specific session.
    • Expanded event timeline examples and updated supported discovery options.

Automated mirror of cli/skill/. Source of truth: FailproofAI/agenteye — do not hand-edit.
@NiveditJain NiveditJain added the skill-sync-agenteye-cli Sync the agenteye-cli skill from source! label Jul 15, 2026
@NiveditJain NiveditJain requested a review from SiddarthAA July 15, 2026 06:20
@coderabbitai

coderabbitai Bot commented Jul 15, 2026

Copy link
Copy Markdown

Review Change Stack

📝 Walkthrough

Walkthrough

Updated AgentEye CLI skill and command-reference documentation to clarify usage, telemetry controls, time validation, event summary versus raw payload retrieval, error output, and supported discovery kinds.

Changes

AgentEye CLI documentation

Layer / File(s) Summary
CLI usage and validation guidance
skills/agenteye-cli/SKILL.md, skills/agenteye-cli/references/commands.md
Clarifies when to use the CLI, telemetry controls, accepted time filters, usage errors, error output, and supported list <kind> values.
Event feed and payload guidance
skills/agenteye-cli/SKILL.md, skills/agenteye-cli/references/commands.md
Documents payload-free event summaries, opt-in raw payload retrieval through --full or --fields payload, session bounds, performance behavior, and retrieval examples.

Estimated code review effort: 1 (Trivial) | ~5 minutes

Suggested reviewers: siddarthaa

Poem

A rabbit read the event trail bright,
With light summaries hopping in sight.
Raw payloads stayed tucked away,
Until bounded flags said “okay!”
CLI docs now guide the way.

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title clearly states the automated sync of the agenteye-cli skill and includes the source commit.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

@coderabbitai coderabbitai 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.

Actionable comments posted: 2

🤖 Prompt for all review comments with AI agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.

Inline comments:
In `@skills/agenteye-cli/references/commands.md`:
- Line 68: Make all raw-payload examples explicit about result caps: in
skills/agenteye-cli/references/commands.md lines 68-68, add an explicit --limit
or label the --all example as capped; in lines 76-78, add limits to the generic,
single-event, and error examples and avoid describing capped queries as the
“whole session”; in skills/agenteye-cli/SKILL.md lines 170-170, add --limit <N>
or direct readers to the exact by-id query.
- Around line 65-67: Align the documented scope of --fields with its event/error
usage: in skills/agenteye-cli/references/commands.md lines 65-67 and 72-73 and
skills/agenteye-cli/SKILL.md lines 115 and 173-174, either include events in the
shared --fields scope or remove the event-specific --fields payload guidance,
applying the same choice consistently at every listed site.
🪄 Autofix (Beta)

Fix all unresolved CodeRabbit comments on this PR:

  • Push a commit to this branch (recommended)
  • Create a new PR with the fixes

ℹ️ Review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

Run ID: 7c7f7fd9-e7c0-444b-b034-fa1814662e93

📥 Commits

Reviewing files that changed from the base of the PR and between 37d6892 and 6f4b1c8.

📒 Files selected for processing (2)
  • skills/agenteye-cli/SKILL.md
  • skills/agenteye-cli/references/commands.md

Comment on lines +65 to +67
free-text term, so broad searches can still be expensive. To get the raw `payload`, opt
into the **full feed** with `--full` (or `--fields payload`) — this hits the heavy
`/api/events`, which is slow at scale, so keep it bounded (pair `--full` with one

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

🎯 Functional Correctness | 🟡 Minor | ⚡ Quick win

🧩 Analysis chain

🏁 Script executed:

#!/bin/bash
set -euo pipefail

printf '\n== files ==\n'
git ls-files 'skills/agenteye-cli/*' | sed 's#^`#-` #'

printf '\n== search flags ==\n'
rg -n --hidden --glob '!**/.git/**' --glob '!*node_modules*' --glob '!*dist*' --glob '!*build*' \
  -- '--fields|--full|--limit|events|errors|session-id|payload' skills/agenteye-cli

printf '\n== outline key docs ==\n'
ast-grep outline skills/agenteye-cli/SKILL.md || true
ast-grep outline skills/agenteye-cli/references/commands.md || true

Repository: FailproofAI/skills

Length of output: 11271


🏁 Script executed:

#!/bin/bash
set -euo pipefail

printf '\n== commands doc excerpt ==\n'
sed -n '34,45p;59,90p' skills/agenteye-cli/references/commands.md | cat -n

printf '\n== skill doc excerpt ==\n'
sed -n '108,185p' skills/agenteye-cli/SKILL.md | cat -n

printf '\n== search for command contract / parser ==\n'
rg -n --hidden --glob '!**/.git/**' --glob '!*node_modules*' --glob '!*dist*' --glob '!*build*' \
  -e 'fields payload|--fields payload|where supported: sessions/evals, keys, query list|events .*--fields|errors .*--fields|--fields .*payload' .

Repository: FailproofAI/skills

Length of output: 12441


Align --fields payload across the docs. references/commands.md scopes --fields to sessions/evals, keys, and query list, but the events/errors guidance also advertises --fields payload. Either add events to the shared --fields scope or remove that event-specific wording.

📍 Affects 2 files
  • skills/agenteye-cli/references/commands.md#L65-L67 (this comment)
  • skills/agenteye-cli/SKILL.md#L115-L115
  • skills/agenteye-cli/SKILL.md#L173-L174
  • skills/agenteye-cli/references/commands.md#L72-L73
🤖 Prompt for AI Agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.

In `@skills/agenteye-cli/references/commands.md` around lines 65 - 67, Align the
documented scope of --fields with its event/error usage: in
skills/agenteye-cli/references/commands.md lines 65-67 and 72-73 and
skills/agenteye-cli/SKILL.md lines 115 and 173-174, either include events in the
shared --fields scope or remove the event-specific --fields payload guidance,
applying the same choice consistently at every listed site.

free-text term, so broad searches can still be expensive. To get the raw `payload`, opt
into the **full feed** with `--full` (or `--fields payload`) — this hits the heavy
`/api/events`, which is slow at scale, so keep it bounded (pair `--full` with one
`--session-id`). e.g. `agenteye --json events --full --session-id run-1 --all | jq '.events[].payload'`.

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

🎯 Functional Correctness | 🟠 Major | ⚡ Quick win

Make raw-payload examples explicit about result caps. Bare --all defaults to 50 rows, and --limit 1000 is still not guaranteed to cover an unusually large session.

  • skills/agenteye-cli/references/commands.md#L68-L68: add an explicit limit or label the example as capped.
  • skills/agenteye-cli/references/commands.md#L76-L78: add limits to the generic, single-event, and error examples, and avoid calling a capped query “whole session.”
  • skills/agenteye-cli/SKILL.md#L170-L170: add --limit <N> or direct readers to the exact by-id query.
📍 Affects 2 files
  • skills/agenteye-cli/references/commands.md#L68-L68 (this comment)
  • skills/agenteye-cli/references/commands.md#L76-L78
  • skills/agenteye-cli/SKILL.md#L170-L170
🤖 Prompt for AI Agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.

In `@skills/agenteye-cli/references/commands.md` at line 68, Make all raw-payload
examples explicit about result caps: in
skills/agenteye-cli/references/commands.md lines 68-68, add an explicit --limit
or label the --all example as capped; in lines 76-78, add limits to the generic,
single-event, and error examples and avoid describing capped queries as the
“whole session”; in skills/agenteye-cli/SKILL.md lines 170-170, add --limit <N>
or direct readers to the exact by-id query.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

skill-sync-agenteye-cli Sync the agenteye-cli skill from source!

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants