Skip to content

feat: add ai-samples skeleton with use-case feature folders#1062

Draft
sedanah-m wants to merge 5 commits into
masterfrom
feat/ai-samples
Draft

feat: add ai-samples skeleton with use-case feature folders#1062
sedanah-m wants to merge 5 commits into
masterfrom
feat/ai-samples

Conversation

@sedanah-m

Copy link
Copy Markdown
Contributor

Add a new ai/ai-samples/ app as a sibling to the existing ai-react-app

@gemini-code-assist gemini-code-assist Bot 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.

Code Review

This pull request introduces a new React-based project structure under ai/ai-samples to demonstrate modular Firebase AI capabilities such as text generation, chat, and image generation. The feedback recommends aligning the firebase dependency version with sibling packages, replacing commented-out Firebase configuration with active placeholder values, and renaming generic Feature components to descriptive names (e.g., ChatFeature, TextGenerationFeature) to improve debuggability in React DevTools.

Comment thread ai/ai-samples/package.json
Comment thread ai/ai-samples/src/config/firebase-config.ts
Comment thread ai/ai-samples/src/features/chat/index.tsx Outdated
Comment thread ai/ai-samples/src/features/function-calling/index.tsx Outdated
Comment thread ai/ai-samples/src/features/image-generation/index.tsx Outdated
Comment thread ai/ai-samples/src/features/multimodal/index.tsx Outdated
Comment thread ai/ai-samples/src/features/structured-output/index.tsx
Comment thread ai/ai-samples/src/features/text-generation/index.tsx Outdated
@sedanah-m
sedanah-m force-pushed the feat/ai-samples branch 2 times, most recently from 8c099e9 to 720d7ff Compare June 22, 2026 18:52
sedanah-m and others added 2 commits July 17, 2026 14:21
* feat: add text-generation and chat isolated samples

* feat: include skeleton for all features; add decoupled logic/scripts

* fix: handleResetChat in chat index.tsx

* Apply suggestion from @gemini-code-assist[bot]

Co-authored-by: gemini-code-assist[bot] <176961590+gemini-code-assist[bot]@users.noreply.github.com>

* Apply suggestion from @gemini-code-assist[bot]

Co-authored-by: gemini-code-assist[bot] <176961590+gemini-code-assist[bot]@users.noreply.github.com>

* update: README, update correct dependency to npm instead of yarn, include app check logic in firebaseAIService

* fix model update

* Simplifies app check by removing dev enviornment check.

* removes the conditional check for existing Firebase app instances before initialization.

---------

Co-authored-by: gemini-code-assist[bot] <176961590+gemini-code-assist[bot]@users.noreply.github.com>
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