posthog icon indicating copy to clipboard operation
posthog copied to clipboard

feat(ph-ai): create notebook tool

Open kappa90 opened this issue 1 month ago • 6 comments

Problem

New create_notebook tool aimed at the Deep Research flow, now the artifact manager supports notebook artifacts, removed old TipTap backend notebook serializer.

Changes

  • Added a generic artifact type registry that maps content classes to database types (we can now extend the manager with more artifact types), with support for visualizations and notebooks
  • Created a notebook streaming system that shows notebook content as it's being generated
  • Added a create_notebook tool for creating rich documents with embedded visualizations
  • Added support for updating existing artifacts

How did you test this code?

Tests + locally

kappa90 avatar Dec 04 '25 13:12 kappa90

[!WARNING] This pull request is not mergeable via GitHub because a downstack PR is open. Once all requirements are satisfied, merge this PR as a stack on Graphite. Learn more

This stack of pull requests is managed by Graphite. Learn more about stacking.

kappa90 avatar Dec 04 '25 13:12 kappa90

Wiz Scan Summary

Scanner Findings
Vulnerability Finding Vulnerabilities -
Data Finding Sensitive Data 1 Info
Secret Finding Secrets -
IaC Misconfiguration IaC Misconfigurations -
SAST Finding SAST Findings -
Total 1 Info

View scan details in Wiz

To detect these findings earlier in the dev lifecycle, try using Wiz Code VS Code Extension.

wiz-7ad640923b[bot] avatar Dec 04 '25 13:12 wiz-7ad640923b[bot]

Size Change: 0 B

Total Size: 3.52 MB

ℹ️ View Unchanged
Filename Size
frontend/dist/toolbar.js 3.52 MB

compressed-size-action

github-actions[bot] avatar Dec 06 '25 20:12 github-actions[bot]

Visual regression: Storybook UI snapshots updated

Changes: 14 snapshots (6 modified, 8 added, 0 deleted)

What this means:

  • Snapshots have been automatically updated to match current rendering
  • Next CI run will switch to CHECK mode to verify stability
  • If snapshots change again, CHECK mode will fail (indicates flapping)

Next steps:

  • Review the changes to ensure they're intentional
  • Approve if changes match your expectations
  • If unexpected, investigate component rendering

Review snapshot changes →

posthog-bot avatar Dec 06 '25 20:12 posthog-bot

Visual regression: Storybook UI snapshots updated

Changes: 6 snapshots (6 modified, 0 added, 0 deleted)

What this means:

  • Snapshots have been automatically updated to match current rendering
  • Next CI run will switch to CHECK mode to verify stability
  • If snapshots change again, CHECK mode will fail (indicates flapping)

Next steps:

  • Review the changes to ensure they're intentional
  • Approve if changes match your expectations
  • If unexpected, investigate component rendering

Review snapshot changes →

posthog-bot avatar Dec 06 '25 21:12 posthog-bot

Query snapshots: Backend query snapshots updated

Changes: 1 snapshots (1 modified, 0 added, 0 deleted)

What this means:

  • Query snapshots have been automatically updated to match current output
  • These changes reflect modifications to database queries or schema

Next steps:

  • Review the query changes to ensure they're intentional
  • If unexpected, investigate what caused the query to change

Review snapshot changes →

posthog-bot avatar Dec 07 '25 11:12 posthog-bot