Skip to content

fix: defer smart composer enrollment#6134

Merged
rebelchris merged 2 commits into
mainfrom
codex/smart-composer-intent-enrollment
Jun 2, 2026
Merged

fix: defer smart composer enrollment#6134
rebelchris merged 2 commits into
mainfrom
codex/smart-composer-intent-enrollment

Conversation

@rebelchris
Copy link
Copy Markdown
Contributor

@rebelchris rebelchris commented Jun 2, 2026

Summary

  • defer smart_composer evaluation until create/edit/share intent instead of button render
  • emit a generic create post outcome event from both control and smart composer creation paths
  • add regression coverage for intent-time evaluation and post-created logging

Tests

  • node ./scripts/typecheck-strict-changed.js
  • NODE_ENV=test pnpm --filter @dailydotdev/shared exec jest src/hooks/post/useSmartComposer.spec.tsx src/hooks/squads/usePostToSquad.spec.tsx src/features/squads/hooks/useMultipleSourcePost.spec.tsx src/components/modals/post/SmartComposerModal.spec.tsx --runInBand
  • pnpm --filter @dailydotdev/shared exec eslint --max-warnings 0

Preview domain

https://codex-smart-composer-intent-enro.preview.app.daily.dev

@vercel
Copy link
Copy Markdown

vercel Bot commented Jun 2, 2026

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
daily-webapp Ready Ready Preview Jun 2, 2026 1:41pm
1 Skipped Deployment
Project Deployment Actions Updated (UTC)
storybook Ignored Ignored Jun 2, 2026 1:41pm

Request Review

@rebelchris rebelchris requested review from capJavert and idoshamun June 2, 2026 13:35
@rebelchris rebelchris merged commit e6db040 into main Jun 2, 2026
13 checks passed
@rebelchris rebelchris deleted the codex/smart-composer-intent-enrollment branch June 2, 2026 13:50
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.

2 participants