Skip to content

Add Q as a UI component - #360

Open
aditya-damerla128 wants to merge 2 commits into
mieweb:mainfrom
aditya-damerla128:feat/q-chat
Open

Add Q as a UI component#360
aditya-damerla128 wants to merge 2 commits into
mieweb:mainfrom
aditya-damerla128:feat/q-chat

Conversation

@aditya-damerla128

Copy link
Copy Markdown
Contributor

Summary

Closes #359.

Adds Q as a UI component backed by @mieweb/q, with a dedicated @mieweb/ui/q entry point so consumers can import Q directly from the UI package.

Changes

  • Add Q component wrapper around @mieweb/q's AgentConfigGenerator.
  • Add dedicated @mieweb/ui/q package export and tsup entry.
  • Re-export the useful Q package surface from the Q entry point.
  • Add Q presets:
    • full Twilio agent schema
    • agent-only schema that hides Twilio/telephony fields
  • Add Storybook examples for the Q component and presets.
  • Add a local type shim for @mieweb/q because the published package declares types but does not include the referenced .d.ts file.

Checks

  • pnpm run typecheck
  • pnpm run lint
  • pnpm run test
  • pnpm run build
  • pnpm run format
  • git diff --check -- package.json pnpm-lock.yaml tsup.config.ts src/q.ts src/components/Q/Q.tsx src/components/Q/Q.stories.tsx src/components/Q/index.ts src/components/Q/mieweb-q.d.ts src/components/Q/presets.ts
  • pnpm run build-storybook

Copilot AI lite review requested due to automatic review settings August 7, 2026 15:57

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Adds a new first-class Q surface to @mieweb/ui by wrapping the @mieweb/q agent config generator and exposing it via a dedicated @mieweb/ui/q entry point, plus presets and Storybook examples.

Changes:

  • Add Q wrapper component and presets (qAgentSchema, qTwilioAgentSchema, config factories).
  • Add @mieweb/ui/q export surface + tsup entry for a dedicated bundle boundary.
  • Add a local TypeScript shim for @mieweb/q and wire in the @mieweb/q dependency.

Reviewed changes

Copilot reviewed 7 out of 9 changed files in this pull request and generated 3 comments.

Show a summary per file
File Description
tsup.config.ts Adds q and components/Q build entries; externalizes @mieweb/q.
src/q.ts New @mieweb/ui/q entry re-exporting Q surface.
src/components/Q/Q.tsx Adds Q wrapper around @mieweb/q’s AgentConfigGenerator.
src/components/Q/Q.stories.tsx Adds Storybook examples for Q and presets.
src/components/Q/presets.ts Adds Twilio + agent-only preset schemas and default configs.
src/components/Q/mieweb-q.d.ts Adds local TS module declarations for @mieweb/q (+ /universal and css).
src/components/Q/index.ts Defines the public re-export surface for @mieweb/ui/q.
package.json Adds ./q export and introduces @mieweb/q as peer/dev dependency.
pnpm-lock.yaml Locks @mieweb/q@1.0.0 and its dependency tree.
Files not reviewed (1)
  • pnpm-lock.yaml: Generated file

Comment thread package.json Outdated
Comment thread pnpm-lock.yaml
Comment thread src/components/Q/Q.tsx
Copilot AI review requested due to automatic review settings August 7, 2026 17:45

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Copilot reviewed 7 out of 9 changed files in this pull request and generated no new comments.

Files not reviewed (1)
  • pnpm-lock.yaml: Generated file
Suppressed comments (1)

pnpm-lock.yaml:7077

  • @mieweb/q pulls in @mieweb/ui@0.2.4 plus lucide-react@0.344.0, whose peer range is react: ^16.5.1 || ^17 || ^18 (but this repo is on React 19). This can lead to peer-dependency warnings/failures in consumer installs (especially with strict peer deps) and potentially duplicate UI copies at runtime.

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.

Add Q as a UI component

2 participants