Skip to content

docs(loops): do not claim the egress blocker is resolved - #25

Merged
drakulavich merged 1 commit into
mainfrom
fix/egress-note-overclaim
Jul 25, 2026
Merged

docs(loops): do not claim the egress blocker is resolved#25
drakulavich merged 1 commit into
mainfrom
fix/egress-note-overclaim

Conversation

@drakulavich

Copy link
Copy Markdown
Owner

Corrects an overstatement I introduced in #24, which merged before this fix was pushed.

The error

#24 declared:

Egress limitation — RESOLVED

cloud.ouraring.com is reachable again; the 403 on CONNECT recorded on 2026-07-17 is gone. No allowlist action is outstanding.

The evidence behind that was a successful spec fetch during the 2026-07-25 run — but that run executed in a local CLI session on a laptop, with no egress proxy configured (CLAUDE_CODE_ENTRYPOINT=cli, no *_proxy variables). The 403s in #21 came from the sandboxed scheduled environment's egress proxy.

Those are different network paths. Reaching the host from a laptop is no evidence at all about the sandbox's allowlist, which is what #21 is actually about.

The practical risk: the note as written justified closing #21 on a check that never touched the blocked path, which would have left the weekly watcher silently unable to do its job — the exact failure #21 exists to prevent.

The correction

  • Section retitled STILL OPEN (api-drift-watcher blocked: Oura spec host not on egress allowlist #21), with the reasoning spelled out and the closing condition named: a scheduled run that reaches cloud.ouraring.com.
  • The Latest run header and the history row now record which environment the run executed in, so a future reader can tell which runs exercised the proxy path and which did not.

No other content from #24 is touched — the drift findings, the field-hash repair, and the notes stay as merged.

Status of #21

Should stay open. Nothing in this session verified the sandbox allowlist, and nothing in the repo configures it — the watcher has no GitHub Action (.github/workflows/ holds only ci.yml and release.yml), so it runs as a Claude Code schedule and that environment's policy is set outside this repo.

The 2026-07-25 run fetched the spec from a local CLI session with no egress
proxy configured. The 403 in #21 came from the sandboxed scheduled
environment's proxy — different network path, so reaching the host from a
laptop says nothing about that allowlist. Calling the limitation RESOLVED
overstated the evidence and would have justified closing #21 on a check
that never exercised the blocked path.

Reworded to STILL OPEN with the closing condition named: a scheduled run
that reaches cloud.ouraring.com. The run environment is now recorded in the
Latest run header and the history row, so a future reader can tell which
sessions actually went through the proxy.

Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01DRnTxyiiF3XHiyFqZJ3soz
@greptile-apps

greptile-apps Bot commented Jul 25, 2026

Copy link
Copy Markdown

Greptile Summary

Corrects the watcher state documentation to keep the sandbox egress limitation open.

  • Identifies the successful 2026-07-25 spec fetch as originating from a local CLI session rather than the scheduled sandbox.
  • Clarifies that issue api-drift-watcher blocked: Oura spec host not on egress allowlist #21 should remain open until a scheduled run reaches the host.
  • Records the execution environment in the latest-run details and history table.

Confidence Score: 5/5

The documentation-only correction appears safe to merge.

The updated text consistently identifies the verified run as local, preserves the unresolved sandbox egress limitation, and states a concrete closing condition for issue #21.

Important Files Changed

Filename Overview
docs/loops/api-drift-watcher-state.md Accurately distinguishes local and scheduled network paths and removes the unsupported claim that sandbox egress was restored.

Reviews (1): Last reviewed commit: "docs(loops): do not claim the egress blo..." | Re-trigger Greptile

@drakulavich
drakulavich merged commit 0e9993a into main Jul 25, 2026
3 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