Skip to content

docs(agents): check serverless-components before adding new code - #1424

Open
lym953 wants to merge 1 commit into
mainfrom
yiming.luo/agents-md-serverless-components
Open

lym953 wants to merge 1 commit into
mainfrom
yiming.luo/agents-md-serverless-components

Conversation

@lym953

@lym953 lym953 commented Oct 14, 2026 •

Copy link
Copy Markdown
Contributor

Overview

The serverless-components repo holds Rust crates that this extension shares with the Azure and Google Cloud Platform (GCP) serverless agents. Agents that work in this repo do not know this. They can write code here that already exists there, or that other agents can also use. For example, #1418 sends oversized trace payloads alone. It might reuse some code from DataDog/serverless-components#154, which splits oversized trace payloads into smaller chunks.

This PR adds a "Where New Code Goes" section to AGENTS.md. It tells agents to do these steps:

  • Check serverless-components before they add a new functionality.
  • Put code that other serverless agents can use in serverless-components.
  • Tell the engineer to speak with the Serverless Azure and GCP team when the decision is not clear. Do the same for a change to an existing crate there.

Testing

None. This PR changes documentation only.

🤖 Generated with Claude Code

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@datadog-prod-us1-6

datadog-prod-us1-6 Bot commented Oct 14, 2026 •

Copy link
Copy Markdown

Pipelines

✨ Unblock PR with BitsAI

❌ Errors

Your PR has failed checks. Please review the issues below and take necessary action before merging.

🚦 2 Pipeline jobs failed

DataDog/datadog-lambda-extension | integration-suite: [payload-size] — 🔧 Needs a code fix, caused by this PR

View more details · View in GitLab

Bottlecap (Rust) | Audit

View more details · View in GitHub Actions

Useful? React with 👍 / 👎

This comment will be updated automatically if new data arrives.
🔗 Commit SHA: 12b3f08 | Docs | View more details | Give us feedback!

@lym953
lym953 marked this pull request as ready for review October 14, 2026 18:02
@lym953
lym953 requested a review from a team as a code owner October 14, 2026 18:02
@lym953
lym953 requested review from shreyamalpani and a balanced review from Copilot October 14, 2026 18:02

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.

🟢 Approval recommended

The documentation-only change matches the stated purpose and has no unresolved issues.

0 open findings

What changed in this PR

Adds guidance for reusing shared Rust crates and deciding where new functionality belongs.

Changes:

  • Check serverless-components before adding functionality.
  • Place reusable code in the shared repository.
  • Consult the Serverless Azure and GCP team for unclear placement or existing crate changes.
File Description
AGENTS.md Adds shared-code placement and coordination guidance.

🧠 Review effort: Balanced


💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

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