Skip to content

Agent fixes - #25

Merged
inercia merged 2 commits into
mainfrom
saurin/agent-fixes
Oct 20, 2025
Merged

Agent fixes#25
inercia merged 2 commits into
mainfrom
saurin/agent-fixes

Conversation

@inercia

@inercia inercia commented Oct 20, 2025

Copy link
Copy Markdown
Owner

No description provided.

@inercia
inercia force-pushed the saurin/agent-fixes branch from 87e48a5 to f3e6134 Compare October 20, 2025 13:32
@inercia
inercia requested a review from Copilot October 20, 2025 13:32

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull Request Overview

This PR primarily reduces log verbosity by downgrading many Info/Printf statements to Debug, introduces a Warn logging method, and adds interactive conversation capabilities to the agent runtime and CLI.

  • Add interactive loop to Agent.Run with per-turn user input and a new CagentRuntime.ContinueConversation method
  • Replace numerous Info/Printf logs with Debug, add Logger.Warn, and adjust some messages to better reflect severity
  • Update agent usage documentation to reflect interactive usage and configuration

Reviewed Changes

Copilot reviewed 17 out of 17 changed files in this pull request and generated 5 comments.

Show a summary per file
File Description
pkg/server/server.go Downgrade tool execution log from Info to Debug for less noisy logs.
pkg/common/panic.go Change recovered panic logs to Debug; still logs to stderr.
pkg/common/logging.go Change init logs to Debug and add Warn method on Logger.
pkg/common/constraints.go Downgrade constraint evaluation logs to Debug.
pkg/command/runner_sandbox.go Change operational logs to Debug for sandbox-exec runner.
pkg/command/runner_firejail.go Change operational logs to Debug for firejail runner.
pkg/command/runner_exec.go Change execution logs to Debug and keep behavior; minor messaging tweaks.
pkg/command/runner_docker.go Change Docker execution logs to Debug.
pkg/command/runner.go Downgrade implicit requirements failure log to Debug.
pkg/command/command_exec.go Downgrade tool completion log to Debug.
pkg/command/command.go Downgrade constraint compilation logs to Debug.
pkg/agent/model.go Use Warn when falling back to OpenAI-compatible model class.
pkg/agent/cagent_runtime.go Add ContinueConversation and convert logs to Debug; initialize single-agent runtime.
pkg/agent/agent.go Implement interactive conversation loop, improved exit handling, and better user prompts.
docs/usage-agent.md Update agent usage guide; fix flow and examples, but introduces minor typos.
cmd/exe.go Downgrade "Executing tool" log to Debug.
cmd/agent.go Improve error handling messaging for context cancellations and clarify prompt usage.

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

Comment thread pkg/common/logging.go
Comment thread pkg/common/panic.go Outdated
Comment thread pkg/common/logging.go
Comment thread docs/usage-agent.md Outdated
Comment thread docs/usage-agent.md Outdated
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
@inercia
inercia merged commit 26f52ec into main Oct 20, 2025
4 checks passed
@inercia
inercia deleted the saurin/agent-fixes branch October 20, 2025 14:59
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