Skip to content

docs: C3 — remove download-time skill scan, docs for the new shape - #1371

Draft
gewenyu99 wants to merge 10 commits into
posthog/functional-c2d-enforcementfrom
posthog/functional-c3-consumers
Draft

gewenyu99 wants to merge 10 commits into
posthog/functional-c2d-enforcementfrom
posthog/functional-c3-consumers

Conversation

@gewenyu99

Copy link
Copy Markdown
Collaborator

Replaces #1316, which GitHub marked merged after a mis-stacked push.

Problem

The download-time skill scan duplicated warlock's tool-use hooks. The docs described the old layout.

Change

  • The download-time skill scan and its install triage option are removed. Warlock's tool-use hooks, including the Bash skill-install hook, stay, and the warlock smoke keeps its rule checks.
  • Docs, READMEs and skills describe the current shape: the layer matrix, the per-program contract, TUI program folders, runner contexts and the control API.

Checks

  • Typecheck, lint and 3,554 unit tests pass.
  • Local nine-program snapshot sweep: eight programs complete with exit 0. self-driving stops at the "Connect GitHub" gate because local PostHog has no GitHub App.

Created with PostHog Desktop

scripts/tui-host.no-jest.ts, which the snapshot and agent e2e routes run,
had drifted since Release B: it imported a deleted legacy runner module,
passed a removed session option, and called authenticate and project-data
resolution with old arities. It now starts the TUI with the CLI's host, logs
in through cliAuthHost, and walks composed programs with run-wizard's own
advanceStep over the TUI flow instead of a hand-copied loop.
check-screens.tsx took SettingsConflict from an agent module that no longer
exports it.

Nothing typechecked scripts/, which is how both went unnoticed; the root
project now includes it.

Generated-By: PostHog Desktop
Task-Id: d14e92bb-6ee1-49b5-8502-39cb80079589
Each layer README states its entries, intent and wiring; the headless one
carries the control route, action and setter tables. AGENTS.md lists the
three layers next to agent, shared and programs.

Generated-By: PostHog Desktop
Task-Id: d14e92bb-6ee1-49b5-8502-39cb80079589
…onsumers

Generated-By: PostHog Desktop
Task-Id: d14e92bb-6ee1-49b5-8502-39cb80079589
Agent, programs, TUI and shared roots now hold only index.ts, types.ts,
the README and tsconfig.layer.json. Every other file moves, with its tests,
into a folder named for one job (agent: security, prompt, progress, sdk;
programs: run, host, registry; tui: app, state, flows; shared: config,
posthog, gateway, claude, skills, pricing, run). Imports within a layer stay
relative and cross-layer entries are unchanged, so the move is renames plus
import rewrites. The agent-runner re-export shim is deleted; its three test
importers use the runner directly. Each layer README gains a folder map and
AGENTS.md states the rule.

Generated-By: PostHog Desktop
Task-Id: d14e92bb-6ee1-49b5-8502-39cb80079589
Merge the restacked C2d (4d2bf0b) into C3. The merge has no textual
conflicts. C3 typechecks `scripts/`, which surfaced A3's fault probe
still written against A3-era paths. It now uses the `@agent` entry and
`@agent/types`, supplies inference auth through
`createCiGatewayAuth`, drops the removed switchboard config, and exits
through `@cli/wizard-abort`.

Generated-By: PostHog Desktop
Task-Id: d14e92bb-6ee1-49b5-8502-39cb80079589
Merge the restacked posthog/functional-c3-consumers (5d60e01) and the new base (1f42c38). The tree matches 5d60e01, where typecheck, lint and tests pass.

Generated-By: PostHog Desktop
Task-Id: d14e92bb-6ee1-49b5-8502-39cb80079589
…nize

Generated-By: PostHog Desktop
Task-Id: d14e92bb-6ee1-49b5-8502-39cb80079589

# Conflicts:
#	e2e-tests/mocks/handlers.ts
#	scripts/tui-host.no-jest.ts
#	src/agent/__tests__/run-agent-standalone.test.ts
#	src/agent/runner/README.md
#	src/agent/runner/sequence/linear.ts
#	src/agent/runner/sequence/orchestrator/orchestrator-runner.ts
#	src/agent/runner/shared/types.ts
#	src/agent/sdk/agent-interface.ts
#	src/programs/__tests__/run-agent-legacy.test.ts
#	src/programs/run/run-program.ts
The headless runner marked its store running as soon as it built it, before
serving the control socket. The server's idle gate reads that phase, so
POST /credentials, /detect and /shutdown answered 409 until a run finished,
and a parent could not log in or detect before its first run. A controlled
store now stays idle until POST /runs sets the phase.

Generated-By: PostHog Desktop
Task-Id: d14e92bb-6ee1-49b5-8502-39cb80079589
Merge the joined C2d (486061f) into C3, which already holds the
organize change from the former C4. B's content keeps its behavior and
takes C4's paths:

- Conflict hunks and B-added files take the incoming side, with import
  specifiers rewritten through `c4-moves.json`, `vi.mock` paths
  included.
- `ledger-watcher.ts` stays deleted in favor of B's `watch-ledger.ts`.
- `skill-run-cleanup.test.ts` moves next to its module in
  `src/shared/skills/`.
- CODEOWNERS and the README owner table point at `src/tui/decks`.
- The A3 fault probe keeps the `@agent` entry and `createCiGatewayAuth`,
  and adds B's harness normalization.

Typecheck with the layer build and probes, lint, 3504 tests and
`build:ci` with the warlock smoke pass.

Generated-By: PostHog Desktop
Task-Id: d14e92bb-6ee1-49b5-8502-39cb80079589
M16: downloadSkill no longer scans the downloaded skill; the install triage option goes with it. Warlock's tool-use hooks stay. Docs, READMEs and skills describe the layer matrix, the per-program contract, TUI program folders, runner contexts and the control API.

Generated-By: PostHog Desktop
Task-Id: d14e92bb-6ee1-49b5-8502-39cb80079589
@github-actions

Copy link
Copy Markdown

🧙 Wizard CI

Run the Wizard CI and test your changes against wizard-workbench example apps by replying with a GitHub comment using one of the following commands:

Test all apps:

  • /wizard-ci all

Test all apps in a directory:

  • /wizard-ci ai-observability
  • /wizard-ci basic-integration
  • /wizard-ci feature-flags
  • /wizard-ci mcp-analytics
  • /wizard-ci replay-vision
  • /wizard-ci revenue
  • /wizard-ci self-driving
  • /wizard-ci warehouse
  • /wizard-ci warehouse-seeded

Test an individual app:

  • /wizard-ci ai-observability/anthropic
  • /wizard-ci ai-observability/google-adk
  • /wizard-ci ai-observability/groq
Show more apps
  • /wizard-ci ai-observability/manual-capture
  • /wizard-ci ai-observability/openai
  • /wizard-ci ai-observability/openai-agents
  • /wizard-ci ai-observability/opentelemetry
  • /wizard-ci ai-observability/vercel-ai
  • /wizard-ci basic-integration/android
  • /wizard-ci basic-integration/angular
  • /wizard-ci basic-integration/astro
  • /wizard-ci basic-integration/django
  • /wizard-ci basic-integration/fastapi
  • /wizard-ci basic-integration/flask
  • /wizard-ci basic-integration/flutter
  • /wizard-ci basic-integration/javascript-node
  • /wizard-ci basic-integration/javascript-web
  • /wizard-ci basic-integration/laravel
  • /wizard-ci basic-integration/next-js
  • /wizard-ci basic-integration/nuxt
  • /wizard-ci basic-integration/python
  • /wizard-ci basic-integration/rails
  • /wizard-ci basic-integration/react-native
  • /wizard-ci basic-integration/react-router
  • /wizard-ci basic-integration/sveltekit
  • /wizard-ci basic-integration/swift
  • /wizard-ci basic-integration/tanstack-router
  • /wizard-ci basic-integration/tanstack-start
  • /wizard-ci basic-integration/vue
  • /wizard-ci feature-flags/django
  • /wizard-ci feature-flags/next-js
  • /wizard-ci mcp-analytics/custom-dispatcher
  • /wizard-ci mcp-analytics/typescript-sdk
  • /wizard-ci replay-vision/javascript-node
  • /wizard-ci replay-vision/next-js
  • /wizard-ci replay-vision/react-vite
  • /wizard-ci revenue/stripe
  • /wizard-ci self-driving/astro
  • /wizard-ci self-driving/fastapi
  • /wizard-ci self-driving/nuxt
  • /wizard-ci self-driving/react-router
  • /wizard-ci self-driving/sveltekit
  • /wizard-ci warehouse/monorepo-env
  • /wizard-ci warehouse/multi-source-next
  • /wizard-ci warehouse/stripe-node
  • /wizard-ci warehouse/zero-source
  • /wizard-ci warehouse-seeded/next-stripe
  • /wizard-ci warehouse-seeded/next-stripe-declined

Test against a Context Mill branch:

  • /wizard-ci all context-mill:my-branch

Add context-mill:<branch> to any command above to pin the Context Mill branch. It defaults to main.

Results will be posted here when complete.

@gewenyu99
gewenyu99 added this pull request to stack #1372 September 27, 2026 15:28
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