Skip to content

style(sight): apply rustfmt to entire agentsight crate#915

Merged
chengshuyi merged 1 commit into
alibaba:mainfrom
jfeng18:style/sight-rustfmt
Jun 15, 2026
Merged

style(sight): apply rustfmt to entire agentsight crate#915
chengshuyi merged 1 commit into
alibaba:mainfrom
jfeng18:style/sight-rustfmt

Conversation

@jfeng18

@jfeng18 jfeng18 commented Jun 15, 2026

Copy link
Copy Markdown
Collaborator

Summary

Apply cargo fmt to the entire agentsight crate. Pure formatting, zero logic changes. This is step 1 of 3 for the Harness roadmap P0 "CI fmt + clippy":

  1. This PR: fmt cleanup (104 files)
  2. Next: clippy cleanup
  3. Then: CI gate (fmt --check + clippy -D warnings)

Supersedes #726 (closed as stale — was a combined fmt+clippy 102-file PR that couldn't get reviewed).

Why split

A single 100+ file PR mixing fmt + clippy is hard to review. Splitting lets this PR be verified mechanically: cargo fmt is deterministic, the diff is guaranteed to have zero semantic changes.

Test plan

  • cargo fmt --check passes (0 diffs)
  • 573 unit tests pass (local + ECS)
  • Compilation verified

🤖 Generated with Claude Code

Pure formatting, zero logic changes. Prerequisite for adding
cargo fmt --check CI gate per the AgentSight Harness roadmap.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
@jfeng18 jfeng18 requested a review from chengshuyi as a code owner June 15, 2026 03:15
@github-actions github-actions Bot added the component:sight src/agentsight/ label Jun 15, 2026
@chengshuyi chengshuyi merged commit 7c14406 into alibaba:main Jun 15, 2026
15 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

component:sight src/agentsight/

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants