feat(skills): publish every report through publish_handoff, no local files - #306
Draft
gewenyu99 wants to merge 1 commit into
Draft
Conversation
…files The wizard's publish_handoff tool now creates the PostHog notebook itself, so skills stop doing it: no notebooks-create call, no [NOTEBOOK_URL] marker, and no human-facing report file dropped in the project. Every report goes out in one publish_handoff call. The two skills that build their report incrementally — audit and events-audit — keep doing so, into the wizard's scratch directory rather than the project root. That chunking exists because a single sustained generation of a full report drops the streaming connection around the ten-minute mark, and publishing needs the whole document in one argument. events-audit publishes in step 6 rather than step 5, since step 6 is what fills the dashboard callout. The integration-v2 notebook skill goes away — creating the notebook was its only job. Generated-By: PostHog Code Task-Id: 3ab3f6ae-5b80-45a2-849d-ac6efdff3505
🧙 Wizard CIRun 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:
Test all apps in a directory:
Test an individual app:
Show more apps
Results will be posted here when complete. |
gewenyu99
marked this pull request as draft
July 31, 2026 03:34
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Stacked on #303. Pairs with wizard#1047, which makes
publish_handoffcreate the notebook.Every skill that produced a human-facing report now publishes it in one
publish_handoffcall. Nonotebooks-create, no[NOTEBOOK_URL]marker (the wizard deleted it), no.mddropped in the user's project.Migrated: audit and its six leaf variants, events-audit, migrate, self-driving, web-analytics, data-warehouse-source, plus the v1 conclude and v2 report steps that were still making the notebook call themselves.
Two things worth reviewing
The incremental builds are kept, redirected to scratch. audit and events-audit deliberately compose their report with
Write+Editper section, because "a single sustained generation of the full document routinely drops the LLM streaming connection around the 10-minute mark".publish_handoffneeds the whole document in one argument, so those skills now build into.posthog-wizard-cache/<name>-report.md, publish from it, and delete it. That directory is the wizard's own scratch space, not something the user reads.events-audit publishes in step 6, not step 5. Step 6 fills
{{dashboard_callout}}in the report after the dashboard exists, so publishing at the end of step 5 would have shipped an unsubstituted placeholder to the reader.Kept on purpose
posthog-web-analytics-findings.jsonis still written — it's explicitly machine-readable, not a report. Same for.posthog-audit-checks.json,.posthog-events-inventory*.json,.posthog-events.jsonand the.posthog-wizard-cache/inputs.Both audit ledger ids (
write-report,upload-notebook) are still resolved, because the wizard seeds both andaudit_resolve_checkserrors on an unknown id — that keeps the released skills working while wizard#1047 ships first.Removed
integration-v2/notebook— creating the notebook was its only job. Dropped from the report agent'sskillslist too.Checks
npm run test18 files / 137 tests pass.npm run buildclean, 8 plugins / 118 skills. Net −474 lines, mostly thenotebooks-create+ 9×notebook-edit+ verify dance in events-audit that the tool replaces.Created with PostHog Code