Skip to content

[NOT-779] Remove agents from notte-cli - #64

Merged
leo-notte merged 2 commits into
mainfrom
remove-agents
Aug 7, 2026
Merged

[NOT-779] Remove agents from notte-cli#64
leo-notte merged 2 commits into
mainfrom
remove-agents

Conversation

@leo-notte

Copy link
Copy Markdown
Contributor

Summary

  • remove the notte agents command group and its generated flags
  • remove agent state, validation, command tests, integration coverage, and documentation
  • keep session and function workflows intact

Validation

  • git diff --check
  • Go tooling is unavailable in this environment, so the Go test suite was not run.

Co-Authored-By: Quaso <quaso@notte.cc>
@greptile-apps

greptile-apps Bot commented Aug 7, 2026

Copy link
Copy Markdown

Greptile Summary

The PR removes the notte agents command group, generated agent flags, validation, tests, and documentation while retaining session and function workflows.

  • Removes agent command registration and agent-specific helpers.
  • Removes agent state constants and code-generation mappings.
  • Preserves cleanup compatibility for legacy current_agent files.
  • Adds coverage for the removed command boundary and legacy-state cleanup.

Confidence Score: 5/5

The PR appears safe to merge.

No blocking failure remains; the previously reported legacy agent-state cleanup issue is addressed at the current head.

Important Files Changed

Filename Overview
internal/cmd/clear.go The previous cleanup issue is fixed by explicitly deleting the legacy current_agent file while keeping agent functionality removed.
internal/cmd/clear_test.go Adds focused coverage proving legacy agent state is removed and missing files are handled idempotently.
internal/cmd/agents.go Removes the complete agent command group and its persisted-state helpers as intended.
internal/cmd/flags_test.go Adds a public command-boundary test confirming agents are absent while sessions and functions remain registered.
internal/config/config.go Removes agent-specific environment and state-file constants without preventing legacy cleanup.
scripts/gen-flags/parser.go Removes the agent-start endpoint from generated CLI flag configuration.

Reviews (2): Last reviewed commit: "Address CI and PR review feedback" | Re-trigger Greptile

Comment thread internal/cmd/clear.go
- fix the pagination test callback syntax and remaining lint failures
- clear legacy agent state left by older CLI versions
- cover the removed agents command and retained workflows
@leo-notte

Copy link
Copy Markdown
Contributor Author

@greptileai review

@leo-notte
leo-notte merged commit 54d5582 into main Aug 7, 2026
4 checks passed
@giordano-lucas giordano-lucas changed the title Remove agents from notte-cli [NOT-779] Remove agents from notte-cli Aug 10, 2026
@giordano-lucas

Copy link
Copy Markdown
Member

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.

2 participants