Conversation
opencodex-pr-automation
Bot
requested a review
from yansigit
as a code owner
September 2, 2026 00:12
opencodex-pr-automation
Bot
added
the
maintainer-sponsored
|
✅ Deterministic PR hygiene checks passed. |
✅ READY
This pull request is already Ready for Review. |
…listeners When hostname is 0.0.0.0 and tls is configured, port 10100 serves HTTPS-only. proxyIdentityAt used directLocalHttpFetch (http-only TCP) -> timeout against TLS port, causing confirmServiceServing to falsely warn 'no proxy answered'. Added one https:// retry via native fetch with rejectUnauthorized:false per attempt for both proxyIdentityAt (/healthz) and probeReadiness (/readyz). Keeps directLocalHttpFetch bypassing proxy env; https via global fetch. Fixes remote session reconnect loops where repair reported serving:false
fix(tls): add https fallback to health/readiness probes for TLS-only listeners
- add scripts/check-hygiene.mjs reusing pr-hygiene.cjs (stdlib only, no new deps) so contributors catch missing_regression_test locally instead of via CI red - wire bun run check:hygiene into prepush and package.json, enhance HYGIENE_FAILURE_HINTS with local command hint - update PULL_REQUEST_TEMPLATE checklist and AGENTS.md commands to surface the gate - fixes dominant deterministic CI failure (behaviorChanged without test) seen on #209, #184, #183; SKIPPED shard is intentional via changes.ci gate (already documented in ci.yml)
fix(ci): defer version-only promotions
fix(ci): harden hygiene gate with local pre-push check
- Add opt-in Remember on this device (localStorage opencodex-remember-token) with prefill and memory bootstrap, Show/Hide toggle, description/placeholder - Center modal (90vw/480px, radius16, overlay blur fix), hide empty notice-err, focus ring - Bump GUI_SESSION_TTL_MS 5m->30m, credentials same-origin - i18n auth.adminTokenDesc/Placeholder/Show/Hide across 9 locales
…and-polish fix(gui): polish admin token sign-in and enable autofill
ci: rerun failed PR jobs once
fix(gui): restore iCloud Keychain autofill for admin token dialog
feat: add remote server access mode toggle
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
1 participant
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
Promote the verified dev integration branch to main.
Verification
This PR was opened automatically after successful Cross-platform CI on dev.
A human maintainer must review and merge it. This automation never merges main.
Checklist