Skip to content

fix: make proxy config writing idempotent#104

Open
fallintoplace wants to merge 1 commit into
openai:mainfrom
fallintoplace:fix/proxy-config-idempotency
Open

fix: make proxy config writing idempotent#104
fallintoplace wants to merge 1 commit into
openai:mainfrom
fallintoplace:fix/proxy-config-idempotency

Conversation

@fallintoplace
Copy link
Copy Markdown

Summary

  • Replace action-managed proxy config sections instead of adding duplicates on repeated runs.
  • Migrate the previous emitted marker format to the new managed sections.
  • Preserve unrelated settings and propagate non-missing-file read errors.

Why

Repeated config writes could duplicate the root provider selection and proxy provider table when the same home directory was reused. The new writer owns only the action-managed sections and leaves unrelated settings intact.

Tests

  • npx pnpm@10.33.0 run check
  • npx pnpm@10.33.0 test

@github-actions
Copy link
Copy Markdown

github-actions Bot commented May 14, 2026

All contributors have signed the CLA ✍️ ✅
Posted by the CLA Assistant Lite bot.

@fallintoplace
Copy link
Copy Markdown
Author

I have read the CLA Document and I hereby sign the CLA

@fallintoplace
Copy link
Copy Markdown
Author

recheck

github-actions Bot added a commit that referenced this pull request May 14, 2026
@fallintoplace fallintoplace force-pushed the fix/proxy-config-idempotency branch from 2cba906 to d23ad17 Compare May 14, 2026 09:08
@fallintoplace fallintoplace marked this pull request as ready for review May 14, 2026 09:09
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