mirror of
https://github.com/Sea-Haven-Industries/open-swe.git
synced 2026-09-30 16:13:15 +00:00
* Adopt upstream modular webhook skeleton (#1621) Apply the durable-interrupt-dispatch refactor: split the monolithic webapp.py into a thin routing layer plus per-source handlers in webhooks/{github,slack,linear}.py, and add completion.py, dispatch.py, and reconcile.py. Reconcile fork divergence by keeping the Bedrock/ Fireworks cross-provider fallback, the no-agent-attribution prompt policy, the dashboard-handoff re-export, and the Slack channel-info cache. ci_autofix is restored on the new dispatch model in a later commit. Refs: #80 * Port fork webhook security delta onto modular handlers Re-apply the fork's security customizations that #1621 did not carry: Linear webhook replay protection (freshness window on the signed webhookTimestamp), per-repo token-cache binding threaded through the thread token resolvers, the INTERNAL_BOT_LOGINS self-check in the review-finding-reply path, and a user-mapping cache refresh before email resolution on the issue and PR-comment paths (multi-replica staleness). Existing fork security tests pass unchanged. Refs: #80 * Restore CI auto-fix on the modular dispatch model Bring back ci_autofix.py and the ci_monitor graph that #1621 deleted, re-wiring the fork's security-reviewed PR-babysitting onto the new structure: the CI-event, autofix-toggle, and review-feedback handlers move into webhooks/github.py and the github_webhook router re-gains the check_run/check_suite/workflow_run/status routing plus the autofix command and actionable-review branches. Auto-fix runs now dispatch through dispatch_agent_run (durability + completion webhook) while keeping the deliberate batch-while-busy skip-rule via get_thread_active_status. Restore langgraph.json's ci_monitor entry and the fork autofix tests (dispatch mock + import paths re-pointed). Refs: #80 * Reformat and update docs for the modular webhook split Point CLAUDE.md and deploy/MIGRATION.md at the new webhooks/ modules and the dispatch/completion/reconcile contract, and mark the user-mapping cache-refresh fix as applied on the GitHub handlers. Refs: #80 * Restore reject backstop for autofix dispatch A burst of near-simultaneous CI events for one head SHA can slip past the busy-check before the dedupe SHA is recorded, so dispatch the autofix path with multitask_strategy=reject (dev's prior platform default) to drop duplicate concurrent creates instead of letting them interrupt each other. Also make the completion failure-reply dedup claim-then-post and drop the unreachable interrupted branch. --------- Co-authored-by: amoussa1229 <166072409+amoussa1229@users.noreply.github.com>
80 lines
3 KiB
Python
80 lines
3 KiB
Python
"""Tool: ``save_plan``. Record the implementation plan for review.
|
|
|
|
Writes the plan as a real ``plan.md`` file in the sandbox (the artifact the
|
|
agent produces and can re-read) and publishes it to the plan-review page, where
|
|
the user and collaborators read it, comment inline, and approve or request
|
|
changes. Available in plan mode (it does not modify the repository under review).
|
|
"""
|
|
|
|
from __future__ import annotations
|
|
|
|
import logging
|
|
from typing import Any
|
|
|
|
from langgraph.config import get_config
|
|
|
|
from ..dashboard.plan_store import PLAN_STATUS_READY, save_plan_content
|
|
|
|
logger = logging.getLogger(__name__)
|
|
|
|
PLAN_FILE_PATH = "plan.md"
|
|
|
|
|
|
async def save_plan(plan_markdown: str) -> dict[str, Any]:
|
|
"""Write your implementation plan as a markdown file and publish it for review.
|
|
|
|
Use this in plan mode once your plan is ready. The plan is saved as
|
|
``plan.md`` in the sandbox and published to the plan-review page linked in
|
|
the conversation, where the user (the owner) and any collaborators can read
|
|
it, leave inline comments, and then approve it or request changes. Call it
|
|
again to overwrite the plan with a revised version when addressing feedback.
|
|
|
|
Write the plan in standard Markdown — headings, bullet/numbered lists, and
|
|
fenced code blocks all render. Structure it clearly (overview, files to
|
|
change, ordered steps, risks).
|
|
|
|
Args:
|
|
plan_markdown: The full plan, as a Markdown document.
|
|
|
|
Returns:
|
|
``{success: True, path}`` on success, or ``{success: False, error}``.
|
|
"""
|
|
content = plan_markdown.strip()
|
|
if not content:
|
|
return {"success": False, "error": "plan_markdown cannot be empty"}
|
|
|
|
try:
|
|
config = get_config()
|
|
except Exception:
|
|
config = {}
|
|
configurable = config.get("configurable", {}) if isinstance(config, dict) else {}
|
|
thread_id = configurable.get("thread_id") if isinstance(configurable, dict) else None
|
|
if not thread_id:
|
|
return {"success": False, "error": "no thread_id in run config"}
|
|
|
|
try:
|
|
path = await _save(str(thread_id), content)
|
|
except Exception as exc: # noqa: BLE001
|
|
logger.exception("save_plan failed for thread %s", thread_id)
|
|
return {"success": False, "error": f"failed to save plan: {exc}"}
|
|
return {"success": True, "path": path}
|
|
|
|
|
|
async def _save(thread_id: str, content: str) -> str:
|
|
sandbox_path = await _write_to_sandbox(thread_id, content)
|
|
await save_plan_content(thread_id, markdown=content, status=PLAN_STATUS_READY)
|
|
return sandbox_path
|
|
|
|
|
|
async def _write_to_sandbox(thread_id: str, content: str) -> str:
|
|
"""Write ``plan.md`` into the thread's sandbox. Best-effort: a missing sandbox
|
|
must not block publishing the plan to the review page."""
|
|
try:
|
|
from ..utils.sandbox_state import get_sandbox_backend
|
|
|
|
backend = await get_sandbox_backend(thread_id)
|
|
await backend.awrite(PLAN_FILE_PATH, content)
|
|
return PLAN_FILE_PATH
|
|
except Exception:
|
|
logger.warning("Could not write plan.md to sandbox for %s", thread_id, exc_info=True)
|
|
return PLAN_FILE_PATH
|