Skip to content

[G01h] Observe in-flight idle drain without deleting owned work #71

Description

@jjangg96

Goal

Add an experiment-only idle-drain observation path that records real in-flight poll ordering across capacity withdrawal without deleting or replaying owned work.

Parent and execution

Bounded implementation under #1, not a replacement for its Goal or live evidence. Use the single parent G01 goal aligned to the current #1 Goal; do not create a second active goal. Primary route: gpt-luna-max (gpt-5.6-luna / max); explicit user override may select grok-high. Independent review uses gpt-luna-max. One dedicated branch/worktree and focused PR. Parent Agent/status remain untouched.

Dependencies

#60 (closed, merged PR #62). Start from reviewed main cf67d4a or its verified descendant. Native blockedBy must agree before implementation.

Scope and acceptance

  • Use the pinned actions/scaleset v0.4.0 high-level listener with initial capacity one and supported SetMaxRunners(0); preserve ACK-before-acquisition ordering.
  • Establish an observable real request-in-flight boundary for one poll, withdraw capacity while response delivery is held, then observe old-poll ACK/acquisition and the next poll's zero capacity. A pre-call marker or post-response hold alone must never claim server-side in-flight proof; timing misses remain inconclusive. Do not fabricate or rewrite service responses.
  • Emit bounded, secret-free drain observations: ordering, fixed categories, seven nonnegative counters, exact owned set/runner identity when present. Inspect before/after; stale zero, absent runner, matching set or idle callback never authorizes deletion or releases uncertain reservations.
  • Integrate the phase through existing experiment approval/broker/ledger boundaries without silently expanding old authority; retain finite one-shot limits and fail-closed storage/identity/cancellation handling.
  • Identify the narrow approved prerequisite for one already-idle owned canary runner. If an experiment-only pre-provision seam is necessary, propose it for independent review before adding it. No production provider or second worker.
  • Meaningful behavioral failing test precedes minimal implementation, then refactor and relevant boundary/failure tests. Use the actual pinned listener with offline synthetic transport; distinguish offline evidence from live qualification.
  • Focused PR records exact commands/results, red evidence, remaining gaps and rollback. Independent contract/security review and clean exact-current-head Codex review plus CI are required before conditional merge.

Exclusions and live gate

Offline implementation only. No credential access/minting, App/key changes, runner registration, live ScaleSet/session/JIT, workflow creation/dispatch/replay/cancellation, Docker/Lima/context/Keychain/launchd changes or cleanup. No RemoveRunner, busy cancellation, production recovery/store/session rehydration, replacement identity or automatic retry. The existing canary group is preparation, not execution authorization. A future live experiment needs a reviewed immutable SHA, exact selected private repository/workflow/resources and explicit maintainer permission. G01 remains open until its full live evidence is independently verified; this PR cannot close #1.

References

docs/HANDOFF.md, docs/EXECUTION.md, docs/decisions/0002-scaleset-integration.md, docs/SECURITY-DESIGN.md, docs/TEST-STRATEGY.md; parent #1 and PR #62.

Activity

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

No one assigned

    Labels

    agent:luna-maxCanonical primary route: gpt-luna-max (gpt-5.6-luna, max reasoning)

    Type

    No type

    Projects

    • Status
      In progress

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions