Skip to content

Update favicon, manifest icons, dependencies, and minor refactors#22

Draft
vdimarco wants to merge 7 commits into
mainfrom
terragon/update-favicon-manifest-w0y6om
Draft

Update favicon, manifest icons, dependencies, and minor refactors#22
vdimarco wants to merge 7 commits into
mainfrom
terragon/update-favicon-manifest-w0y6om

Conversation

@vdimarco

@vdimarco vdimarco commented Oct 24, 2025

Copy link
Copy Markdown

Summary

  • Replaced app favicon with updated asset
  • Updated PWA manifest icons to align with the new favicon
  • Added binary-extensions, chalk, @radix-ui/react-tooltip, @radix-ui/react-visually-hidden dependencies to apps/sim to support binary assets and improved UI tooling
  • Updated docs app dependencies to support MDX processing
  • Extended Next.js config (apps/sim/next.config.ts) to allow serverExternalPackages for test tooling (pdf-parse, playwright, playwright-core, @browserbasehq/stagehand, chromium-bidi)
  • Minor internal refactors: reorganized gatewayz provider imports and utils imports for consistency (no user-facing changes)
  • Small test adjustments to ensure consistency after refactors
  • bun.lock updated accordingly to reflect new dependencies

Changes

Assets

  • favicon.ico: Replaced with updated icon
  • Updated manifest icons to reference the new favicon where applicable

Dependencies

  • apps/sim/package.json: Added "binary-extensions", "chalk", "@radix-ui/react-tooltip", "@radix-ui/react-visually-hidden" dependencies
  • apps/docs/package.json: Added "remark" and "remark-mdx" dev dependencies
  • bun.lock: Updated to include new dependencies
  • Note: bun.lock updated accordingly to reflect new dependencies

Code Refactors

  • apps/sim/providers/gatewayz/index.ts: Minor import reordering
  • apps/sim/providers/utils.ts: Reordered imports; ensure no functional changes
  • apps/sim/next.config.ts: Extended serverExternalPackages to include pdf-parse, playwright, playwright-core, @browserbasehq/stagehand, chromium-bidi

Tests

  • apps/sim/executor/index.test.ts: Adjusted test formatting to align with refactor changes

Notes

  • This is an assets-and-dependency-change; no application logic changes.

Test plan

  • Build and run; verify favicon appears in browser tab
  • Check PWA manifest icons render correctly in install prompts and manifest contexts
  • Ensure no regressions in app loading or branding

🌿 Generated by Terry


ℹ️ Tag @terragon-labs to ask questions and address PR feedback

📎 Task: https://www.terragonlabs.com/task/84a95217-6a13-4b45-bfd4-d4c3bc53b62f

Updated the root-level favicon.ico in the app directory to use the new
purple "H" logo. In Next.js, the app/favicon.ico file takes precedence
over public/favicon files.

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude <noreply@anthropic.com>
@coderabbitai

coderabbitai Bot commented Oct 24, 2025

Copy link
Copy Markdown

Important

Review skipped

Draft detected.

Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

✨ Finishing touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch terragon/update-favicon-manifest-w0y6om

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

Added binary-extensions package which is imported in app/api/files/parse/route.ts
but was not declared in package.json, causing test failures.

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude <noreply@anthropic.com>
@vdimarco vdimarco changed the title Update favicon and manifest icons Update favicon, manifest icons, and dependencies Oct 24, 2025
vdimarco and others added 4 commits October 24, 2025 01:21
Ran biome formatter and linter to fix:
- Formatting issues in executor/index.test.ts
- Import ordering in providers files

These were causing build failures in CI.

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude <noreply@anthropic.com>
Added remark and remark-mdx packages which are imported in
apps/docs/lib/llms.ts but were not declared in the docs package.json,
causing build failures.

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude <noreply@anthropic.com>
Added missing dependencies that were causing build failures:
- @radix-ui/react-tooltip
- @radix-ui/react-visually-hidden
- lodash (for lodash/isEqual and lodash/cloneDeep)
- chalk (for console logger)

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude <noreply@anthropic.com>
- Add node-fetch dependency for API route
- Externalize playwright and related packages from webpack bundle
- Prevents bundling errors with playwright server-side dependencies

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude <noreply@anthropic.com>
@vdimarco vdimarco changed the title Update favicon, manifest icons, and dependencies Update favicon, manifest icons, dependencies, and minor refactors Oct 24, 2025
- Add webpack config to exclude dns and util modules from client bundle
- Add @azure/storage-blob to serverExternalPackages
- Fix module resolution for server-only dependencies

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude <noreply@anthropic.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