chore: update AGENTS.md to reflect recent changes - #90
Open
RedStar071 wants to merge 1 commit into
Open
RedStar071 wants to merge 1 commit into
RedStar071 wants to merge 1 commit into
Conversation
Document the OIDC-404 publish retry wrapper added in #88 (scripts/run-with-retry.mjs), which scripts/publish-snapshot.mjs now uses but the tagged-release publish script does not yet. Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_014vf32fx4mkAqvKopnNpjbF
◈ PR LensNote The title starts with
|
RedStar071
changed the title
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
Weekly AGENTS.md maintenance pass, reviewing PRs merged into
mainsince the last run (#81, on 2026-09-07).fix(release): retry changeset publish on OIDC 404 flakiness) addedscripts/run-with-retry.mjsand wired it intoscripts/publish-snapshot.mjs. AGENTS.md already documents the release/publish flow in detail (OIDC trusted publishing,.changeset/README.md), so it's the kind of non-obvious CI behavior this file is meant to capture:changeset publishfires one concurrent OIDC exchange per package and npm sometimes 404s a subset of them, and the snapshot path now retries up to 3 times to absorb that. Also noted that the tagged-releasepnpm run publishpath (used byrelease.ymlviachangesets/action) does not use this retry wrapper yet, sincescripts/publish.mjs(added in the same PR) isn't currently referenced by any script or workflow.Other PRs in the window were reviewed and did not warrant changes:
chore: update changelog and releaseversion bumps (already covered by the existing release-process description).fix(plugin-i18next): split TFunction helpers into real TS overloads) — internal bug fix toplugin-i18next's public functions; doesn't change build/test/dev commands or establish a new documented convention.chore(deps): update vitest monorepo to v5) — bumpsvitest/@vitest/coverage-v8to v5. Checked the newvitestpeerenginesrange (^22.12.0 || ^24.0.0 || >=26.0.0) against AGENTS.md's Node-version notes; the Cloud VM'sv22.14.0still satisfies it, andvitest.config.ts's Vite-decorator gotcha is unaffected, so no change needed there.Test plan
scripts/publish.mjsis unreferenced (grep across the repo) before describing it as not-yet-wired-in.🤖 Generated with Claude Code
https://claude.ai/code/session_014vf32fx4mkAqvKopnNpjbF
Generated by Claude Code