Skip to content

Conversation

@sestinj
Copy link
Contributor

@sestinj sestinj commented Jan 26, 2026

Summary

  • Adds info-level logging for all metadata endpoint requests in cn CLI
  • Logs include full URL, agent ID, metadata keys, request body, response status, and timing (duration in ms)
  • Uses [metadata] prefix for easy filtering in logs

Test plan

  • Run cn with an agent ID and verify metadata requests are logged to ~/.continue/logs/cn.log
  • Filter logs with grep "[metadata]" ~/.continue/logs/cn.log to verify prefix works
  • Verify timing information is accurate

🤖 Generated with Claude Code


Continue Tasks: ▶️ 3 queued — View all


Summary by cubic

Adds info-level logging for metadata endpoint requests in the cn CLI to improve debugging and visibility. Logs include full URL, agent ID, metadata keys, request body, response status, and duration, with a [metadata] prefix for easy filtering.

  • New Features

    • Logs POST agents/{id}/metadata with full URL, status, and timing (ms).
    • Includes agentId, metadata keys, and request body.
    • Clear messages for auth required and API errors, all prefixed with [metadata].
  • Refactors

    • Moved interruption cleanup and agent completion check to helpers for reuse and clarity.

Written for commit 737ac6f. Summary will update on new commits.

@sestinj sestinj requested a review from a team as a code owner January 26, 2026 07:17
@sestinj sestinj requested review from RomneyDa and removed request for a team January 26, 2026 07:17
@dosubot dosubot bot added the size:M This PR changes 30-99 lines, ignoring generated files. label Jan 26, 2026
@github-actions
Copy link

⚠️ PR Title Format

Your PR title doesn't follow the conventional commit format, but this won't block your PR from being merged. We recommend using this format for better project organization.

Expected Format:

<type>[optional scope]: <description>

Examples:

  • feat: add changelog generation support
  • fix: resolve login redirect issue
  • docs: update README with new instructions
  • chore: update dependencies

Valid Types:

feat, fix, docs, style, refactor, perf, test, build, ci, chore, revert

This helps with:

  • 📝 Automatic changelog generation
  • 🚀 Automated semantic versioning
  • 📊 Better project history tracking

This is a non-blocking warning - your PR can still be merged without fixing this.

@github-actions
Copy link

github-actions bot commented Jan 26, 2026

✅ Review Complete

Code Review Summary

⚠️ Continue API authentication failed. Please check your CONTINUE_API_KEY.


Improves debugging capabilities for the metadata endpoint by logging
at info level with full URL, timing, request body, and response status.

Co-Authored-By: Claude Opus 4.5 <[email protected]>
@dosubot dosubot bot added size:L This PR changes 100-499 lines, ignoring generated files. and removed size:M This PR changes 30-99 lines, ignoring generated files. labels Jan 26, 2026
Copy link
Contributor

@cubic-dev-ai cubic-dev-ai bot left a comment

Choose a reason for hiding this comment

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

No issues found across 1 file

@sestinj sestinj merged commit 77e5af8 into main Jan 26, 2026
55 checks passed
@sestinj sestinj deleted the nate/fixes-4888 branch January 26, 2026 07:30
@github-project-automation github-project-automation bot moved this from Todo to Done in Issues and PRs Jan 26, 2026
@github-actions github-actions bot locked and limited conversation to collaborators Jan 26, 2026
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

size:L This PR changes 100-499 lines, ignoring generated files.

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

2 participants