Skip to content

Merge main into fix-mobile-download, resolve conflicts#295

Open
DHCross wants to merge 3 commits intomainfrom
fix-mobile-download
Open

Merge main into fix-mobile-download, resolve conflicts#295
DHCross wants to merge 3 commits intomainfrom
fix-mobile-download

Conversation

@DHCross
Copy link
Owner

@DHCross DHCross commented Nov 11, 2025

No description provided.

@netlify
Copy link

netlify bot commented Nov 11, 2025

Deploy Preview for sprightly-genie-998c07 ready!

Name Link
🔨 Latest commit 760eb41
🔍 Latest deploy log https://app.netlify.com/projects/sprightly-genie-998c07/deploys/6912924e0531e80008692d92
😎 Deploy Preview https://deploy-preview-295--sprightly-genie-998c07.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.
Lighthouse
Lighthouse
1 paths audited
Performance: 52 (🔴 down 2 from production)
Accessibility: 97 (no change from production)
Best Practices: 92 (🔴 down 8 from production)
SEO: 100 (no change from production)
PWA: -
View the detailed breakdown and full score reports

To edit notification comments on pull requests, go to your Netlify project configuration.

Copy link

@chatgpt-codex-connector chatgpt-codex-connector bot left a comment

Choose a reason for hiding this comment

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

💡 Codex Review

Here are some automated review suggestions for this pull request.

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

Comment on lines 53 to 55
"dependencies": {
"@google/generative-ai": "^0.24.1",
"chart.js": "^4.5.0",
"chartjs-plugin-annotation": "^3.1.0",

Choose a reason for hiding this comment

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

P1 Badge Removing @google/generative-ai leaves unresolved import

package.json no longer declares @google/generative-ai, but lib/llm.ts still imports GoogleGenerativeAI from that package. Running the usual install/build (e.g. npm install && npm run build) will now fail with “Cannot find module '@google/generative-ai'” because the module will not be installed. Unless the Gemini integration is removed elsewhere, this dependency needs to stay in the manifest to keep the build working.

Useful? React with 👍 / 👎.

Copy link
Owner Author

Choose a reason for hiding this comment

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

@copilot I no longer use Gemini for the chatbot. I use Perplexity API. That's the module isn't there.

Copy link
Contributor

Copilot AI commented Nov 11, 2025

@DHCross I've opened a new pull request, #298, to work on those changes. Once the pull request is ready, I'll request review from you.

Copilot AI and others added 2 commits November 10, 2025 19:31
Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com>
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This PR merges the main branch into fix-mobile-download and resolves dependency conflicts. The primary change removes the obsolete @google/generative-ai dependency, which was already eliminated during the October 2025 migration from Gemini to Perplexity API for the Poetic Brain component.

  • Removes unused @google/generative-ai package dependency

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.

3 participants