Skip to content

Fix/client bundle server leak - #901

Merged
shanoysinc merged 2 commits into
colorstackorg:mainfrom
Norus-Tech:fix/client-bundle-server-leak
Aug 13, 2026
Merged

Fix/client bundle server leak#901
shanoysinc merged 2 commits into
colorstackorg:mainfrom
Norus-Tech:fix/client-bundle-server-leak

Conversation

@shanoysinc

Copy link
Copy Markdown
Collaborator

Description ✏️

Adds admin company management tooling and fixes two LinkedIn-related issues discovered during testing.

  • Move parseLinkedInCompanySlug into employment.utils.ts so employment.types.ts no longer imports server-only use cases (prevents Redis/Sentry Node SDK from leaking into the client bundle)
  • Fall back to the largest logo in Apify's logos array when the top-level logo field is missing

Type of Change 🐞

  • Feature - A non-breaking change which adds functionality.
  • Fix - A non-breaking change which fixes an issue.
  • Refactor - A change that neither fixes a bug nor adds a feature.
  • Documentation - A change only to in-code or markdown documentation.
  • Tests - A change that adds missing unit/integration tests.
  • Chore - A change that is likely none of the above.

Checklist ✅

  • I have done a self-review of my code.
  • I have manually tested my code (if applicable).
  • I have added/updated any relevant documentation (if applicable).

@shanoysinc
shanoysinc requested a review from ramiAbdou as a code owner August 13, 2026 17:48
@shanoysinc
shanoysinc merged commit 6ed3ca5 into colorstackorg:main Aug 13, 2026
2 checks passed
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