Skip to content

Conversation

@yurishkuro
Copy link
Member

Summary

  • Remove project structure, setup, build, and test instructions that duplicate content already in CONTRIBUTING.md
  • Add prominent "Required Workflow" section emphasizing that agents must run fmt, lint, and test on the entire repository before completing tasks
  • Reorganize "Do Not Edit" section to include submodules alongside auto-generated files
  • Reference CONTRIBUTING.md for detailed contribution guidelines

Changes

  • Reduced file from 51 lines to 30 lines
  • Removed redundant documentation present in CONTRIBUTING.md
  • Emphasized critical workflow steps that AI agents commonly neglect
  • Improved logical grouping of "Do Not Edit" warnings

🤖 Generated with Claude Code

- Remove project structure, setup, build, and test instructions that
  duplicate content already in CONTRIBUTING.md
- Add prominent "Required Workflow" section emphasizing that agents must
  run fmt, lint, and test on the entire repository before completing tasks
- Reorganize "Do Not Edit" section to include submodules alongside
  auto-generated files
- Reference CONTRIBUTING.md for detailed contribution guidelines

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude <noreply@anthropic.com>
Signed-off-by: Yuri Shkuro <yuri_shkuro@Yuris-MacBook-Pro.local>
@yurishkuro yurishkuro requested a review from a team as a code owner January 15, 2026 18:05
Copy link
Contributor

Copilot AI left a comment

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 streamlines the AGENTS.md file by removing redundant documentation that duplicates content already present in CONTRIBUTING.md, reducing the file from 51 to 31 lines. The changes emphasize critical workflow steps that AI agents commonly neglect and improve the organization of important warnings.

Changes:

  • Removed duplicate project structure, setup, build, and test instructions
  • Added prominent "Required Workflow" section emphasizing mandatory fmt, lint, and test steps
  • Reorganized "Do Not Edit" section to separately list auto-generated files and submodules

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@codecov
Copy link

codecov bot commented Jan 15, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 95.50%. Comparing base (b7250f5) to head (5e68ac2).
⚠️ Report is 1 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #7879      +/-   ##
==========================================
+ Coverage   95.49%   95.50%   +0.01%     
==========================================
  Files         305      305              
  Lines       16212    16212              
==========================================
+ Hits        15481    15483       +2     
+ Misses        572      571       -1     
+ Partials      159      158       -1     
Flag Coverage Δ
badger_v1 9.15% <ø> (ø)
badger_v2 1.93% <ø> (ø)
cassandra-4.x-v1-manual 13.53% <ø> (ø)
cassandra-4.x-v2-auto 1.92% <ø> (ø)
cassandra-4.x-v2-manual 1.92% <ø> (ø)
cassandra-5.x-v1-manual 13.53% <ø> (ø)
cassandra-5.x-v2-auto 1.92% <ø> (ø)
cassandra-5.x-v2-manual 1.92% <ø> (ø)
clickhouse 1.97% <ø> (ø)
elasticsearch-6.x-v1 17.47% <ø> (ø)
elasticsearch-7.x-v1 17.50% <ø> (ø)
elasticsearch-8.x-v1 17.66% <ø> (ø)
elasticsearch-8.x-v2 1.93% <ø> (ø)
elasticsearch-9.x-v2 1.93% <ø> (ø)
grpc_v1 8.81% <ø> (ø)
grpc_v2 1.93% <ø> (ø)
kafka-3.x-v2 1.93% <ø> (ø)
memory_v2 1.93% <ø> (ø)
opensearch-1.x-v1 17.55% <ø> (ø)
opensearch-2.x-v1 17.55% <ø> (ø)
opensearch-2.x-v2 1.93% <ø> (ø)
opensearch-3.x-v2 1.93% <ø> (ø)
query 1.93% <ø> (ø)
tailsampling-processor 0.55% <ø> (ø)
unittests 94.15% <ø> (+0.01%) ⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@github-actions
Copy link

github-actions bot commented Jan 15, 2026

Metrics Comparison Summary

Total changes across all snapshots: 0

Detailed changes per snapshot

summary_metrics_snapshot_cassandra

📊 Metrics Diff Summary

Total Changes: 0

  • 🆕 Added: 0 metrics
  • ❌ Removed: 0 metrics
  • 🔄 Modified: 0 metrics
  • 🚫 Excluded: 106 metrics

summary_metrics_snapshot_cassandra

📊 Metrics Diff Summary

Total Changes: 0

  • 🆕 Added: 0 metrics
  • ❌ Removed: 0 metrics
  • 🔄 Modified: 0 metrics
  • 🚫 Excluded: 106 metrics

summary_metrics_snapshot_cassandra

📊 Metrics Diff Summary

Total Changes: 0

  • 🆕 Added: 0 metrics
  • ❌ Removed: 0 metrics
  • 🔄 Modified: 0 metrics
  • 🚫 Excluded: 53 metrics

➡️ View full metrics file

Copy link
Contributor

@jkowall jkowall left a comment

Choose a reason for hiding this comment

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

Looks good, it seems less explicit (just read contributing.md) but I guess it should do so hopefully.

Signed-off-by: Yuri Shkuro <github@ysh.us>
Copilot AI review requested due to automatic review settings January 15, 2026 19:26
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Copilot reviewed 1 out of 1 changed files in this pull request and generated no new comments.


💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@yurishkuro yurishkuro merged commit 1afbfd9 into jaegertracing:main Jan 15, 2026
59 checks passed
@yurishkuro yurishkuro deleted the streamline-agents-md branch January 15, 2026 19:37
Manik2708 pushed a commit to Manik2708/jaeger that referenced this pull request Jan 20, 2026
## Summary
- Remove project structure, setup, build, and test instructions that
duplicate content already in CONTRIBUTING.md
- Add prominent "Required Workflow" section emphasizing that agents must
run fmt, lint, and test on the entire repository before completing tasks
- Reorganize "Do Not Edit" section to include submodules alongside
auto-generated files
  - Reference CONTRIBUTING.md for detailed contribution guidelines

  ## Changes
  - Reduced file from 51 lines to 30 lines
  - Removed redundant documentation present in CONTRIBUTING.md
  - Emphasized critical workflow steps that AI agents commonly neglect
  - Improved logical grouping of "Do Not Edit" warnings

  🤖 Generated with [Claude Code](https://claude.com/claude-code)

---------

Signed-off-by: Yuri Shkuro <yuri_shkuro@Yuris-MacBook-Pro.local>
Signed-off-by: Yuri Shkuro <github@ysh.us>
Co-authored-by: Yuri Shkuro <yuri_shkuro@Yuris-MacBook-Pro.local>
Co-authored-by: Claude <noreply@anthropic.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants