docs: reflect v2 pivot — agent hosting, credential firewall#7
Conversation
index.mdx: - Update description and intro to v2 positioning - Add "Why we pivoted" section with the full rationale - Replace spec-to-PR feature cards with agent hosting features (always-on, credentials hidden, webhooks, observability, spend ceiling, kill switch) - Remove pivot-in-progress notice from Tip block how-it-works.mdx: - Full rewrite from spec-to-PR lifecycle to agent hosting model - Mermaid diagrams: agent hosting overview, credential firewall sequence, runtime architecture sequence - Step-by-step: connect agent, store credentials, firewall injection, webhooks, observe and control - Spend control table Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
Greptile SummaryThis PR rewrites
Confidence Score: 5/5Safe to merge — all prior P1 concerns resolved; one trivial P2 style nit remains. All previous P1 issues (wrong CLI command, missing Mission Control branding, missing code formatting on zombiectl/zombied) are addressed in this commit. The only remaining finding is a P2 wording issue ('in minutes' time estimate) that does not affect documentation correctness or user paths. No files require special attention. Important Files Changed
Sequence DiagramsequenceDiagram
participant Dev as Developer
participant CLI as `zombiectl`
participant API as `zombied` API
participant Q as Redis Streams
participant W as `zombied` worker
participant S as Sandbox process
participant F as Credential firewall
participant E as External API
Dev->>CLI: deploy agent
Dev->>CLI: zombiectl skill-secret put
CLI->>API: POST /v1/agents (agent config)
API->>Q: enqueue agent_start
Q->>W: claim work
W->>S: spawn sandboxed process
S->>F: outbound request (no credentials)
F->>F: lookup(workspace_id, target_host)
F->>E: forwarded request + injected Bearer token
E-->>F: 200 OK
F-->>S: 200 OK (credential never in sandbox)
F->>F: audit_log.append(action, ts, cost_tokens)
W->>CLI: SSE: status, logs, cost
Prompt To Fix All With AIThis is a comment left during a code review.
Path: index.mdx
Line: 23
Comment:
**Time estimate in user-facing copy**
The phrase "in minutes" is a time estimate, which conflicts with the AGENTS.md rule: *Do not use time estimates or effort ratings in user-facing docs*. Actual setup time varies across agent complexity, credential count, and webhook configuration, so this phrasing could also feel like an unfulfilled promise.
```suggestion
Connect your agent and have it running quickly.
```
**Context Used:** AGENTS.md ([source](https://app.greptile.com/review/custom-context?memory=e994989b-b25e-4468-be5d-6bde3ddb0ce6))
How can I resolve this? If you propose a fix, please make it concise.Reviews (2): Last reviewed commit: "fix: address greptile P1/P2 — correct CL..." | Re-trigger Greptile |
Summary
index.mdx: Updated to v2 positioning. Added "Why we pivoted" rationale (from README). Replaced spec-to-PR feature cards with agent hosting features: always-on agents, credentials hidden, webhooks wired, observability, spend ceiling, kill switch. Removed in-progress pivot notice from Tip.how-it-works.mdx: Full rewrite. Spec-to-PR lifecycle replaced with agent hosting model — credential firewall, always-on execution, webhook routing, observability. Includes mermaid diagrams for the hosting overview, credential firewall sequence, and runtime architecture.Test plan
🤖 Generated with Claude Code