Skip to content

feat(live-graph): add human approval wait state and transient retry p… - #21

Open
varunsood189 wants to merge 5 commits into
theschoolofai:mainfrom
varunsood189:main
Open

varunsood189 wants to merge 5 commits into
theschoolofai:mainfrom
varunsood189:main

Conversation

@varunsood189

@varunsood189 varunsood189 commented Jul 24, 2026 •

Copy link
Copy Markdown

Capability

Users can park high-stakes live-graph tasks in a non-blocking WAITING human-approval state, resume or cancel them with human_approval_granted / human_approval_denied, and apply a RetryPolicy that retries transient failures (for example HTTP 429) while immediately failing permanent errors.

Evidence subsection: README.md → Live Graph Extension: Human Approval & Transient Retry Policy.

Proof

  • I added one evidence subsection to README.md.
  • It contains the exact prompt or API request.
  • It contains the graph and ordered event trace.
  • It contains the actual final result and evidence.
  • It identifies every agent/provider assignment.
  • It shows an adversarial failure before the fix and the same attack failing afterward.
  • It includes commands that reproduce the result from a fresh checkout.

Boundaries

  • glc_v3 still owns all provider credentials and model routing.
  • Memory authorization happens before retrieval.
  • No .env, credentials, personal memory, databases, or unrestricted local paths are committed.
  • uv run ruff check . passes.
  • uv run pytest -q passes.

Test results

Verified from a fresh local checkout of this PR branch:

$ uv run ruff check .
All checks passed!

$ uv run pytest -q
49 passed

Extension-focused reproduction:

uv sync
uv run pytest tests/test_live_graph_extension.py

@theschoolofai theschoolofai left a comment

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Requesting changes: the PR checklist is entirely unchecked and does not provide the required S13 evidence or test results.

Keep the Boundaries checklist honest so PR review can verify lint and tests.

Co-authored-by: Cursor <cursoragent@cursor.com>
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.

2 participants