Skip to content

style(agent-core-v2): fold retry-budget note into the file header - #1742

Merged
sailist merged 1 commit into
MoonshotAI:mainfrom
sailist:style/agent-core-v2-retry-header-comment
Jul 15, 2026
Merged

style(agent-core-v2): fold retry-budget note into the file header#1742
sailist merged 1 commit into
MoonshotAI:mainfrom
sailist:style/agent-core-v2-retry-header-comment

Conversation

@sailist

@sailist sailist commented Jul 15, 2026

Copy link
Copy Markdown
Collaborator

Related Issue

Follow-up to #1740 (Codex review comment).

Problem

The retry-budget comment added in #1740 sits beside an exported constant in packages/agent-core-v2/src/_base/utils/retry.ts, but packages/agent-core-v2/AGENTS.md requires comments to live only in the top-of-file /** */ block, never beside functions, methods, or statements. #1740 was merged before this fix could be pushed.

What changed

Fold the default-budget rationale into the file header and remove the standalone comment. Comment-only change; no behavior difference, no changeset needed.

Checklist

  • I have read the CONTRIBUTING document.
  • I have linked a related issue, or explained the problem above.
  • I have added tests that prove my feature works. (N/A — comment-only change)
  • Ran gen-changesets skill, or this PR needs no changeset.
  • Ran gen-docs skill, or this PR needs no doc update.

packages/agent-core-v2/AGENTS.md requires comments to live only in the
top-of-file block, never beside statements.
@changeset-bot

changeset-bot Bot commented Jul 15, 2026

Copy link
Copy Markdown

⚠️ No Changeset found

Latest commit: 37596d9

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.

This PR includes no changesets

When changesets are added to this PR, you'll see the packages that this PR includes changesets for and the associated semver types

Click here to learn what changesets are, and how to add one.

Click here if you're a maintainer who wants to add a changeset to this PR

@pkg-pr-new

pkg-pr-new Bot commented Jul 15, 2026

Copy link
Copy Markdown
pnpm dlx https://pkg.pr.new/@moonshot-ai/kimi-code@37596d9
npx https://pkg.pr.new/@moonshot-ai/kimi-code@37596d9

commit: 37596d9

@sailist
sailist merged commit 1824e73 into MoonshotAI:main Jul 15, 2026
15 checks passed
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.

1 participant