Skip to content

Fix dependencies#15617

Merged
Weiko merged 1 commit intomainfrom
fix-hacktoberfest-applications
Nov 4, 2025
Merged

Fix dependencies#15617
Weiko merged 1 commit intomainfrom
fix-hacktoberfest-applications

Conversation

@martmull
Copy link
Copy Markdown
Contributor

@martmull martmull commented Nov 4, 2025

As title

Copy link
Copy Markdown
Contributor

@greptile-apps greptile-apps bot left a comment

Choose a reason for hiding this comment

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

Greptile Overview

Greptile Summary

Standardized twenty-sdk dependency declarations across all Twenty apps and CLI templates to use semver ranges (^0.0.3) and ensured all apps have the required dependency.

Key changes:

  • Added missing twenty-sdk ^0.0.3 dependency to 6 apps that were importing from it but lacked the declaration
  • Standardized existing twenty-sdk declarations from exact version (0.0.3) to caret range (^0.0.3) in 5 packages
  • Updated twenty-sdk from ^0.0.2 to ^0.0.3 in ai-meeting-transcript
  • Added missing @types/node ^24.7.2 devDependency to 5 apps for TypeScript support
  • Upgraded @types/node from ^20.0.0 to ^24.7.2 in meeting-transcript for consistency

Confidence Score: 5/5

  • Safe to merge - critical dependency fixes with no breaking changes
  • All changes are straightforward dependency updates that fix missing runtime dependencies and standardize version ranges using proper semver conventions. The apps were already importing from twenty-sdk in their application.config.ts files but lacked the dependency declaration, which would cause runtime errors.
  • No files require special attention

Important Files Changed

File Analysis

Filename Score Overview
packages/twenty-apps/hacktoberfest-2025/last-email-interaction/package.json 5/5 Added missing twenty-sdk ^0.0.3 and @types/node ^24.7.2 dependencies
packages/twenty-apps/hacktoberfest-2025/linkedin-browser-extension/application/package.json 5/5 Added missing twenty-sdk ^0.0.3 dependency
packages/twenty-apps/hacktoberfest-2025/mailchimp-synchronizer/package.json 5/5 Added missing twenty-sdk ^0.0.3 and @types/node ^24.7.2 dependencies
packages/twenty-apps/hacktoberfest-2025/meeting-transcript/package.json 5/5 Added missing twenty-sdk ^0.0.3, updated @types/node from ^20.0.0 to ^24.7.2
packages/twenty-apps/hacktoberfest-2025/rollup-engine/package.json 5/5 Added missing twenty-sdk ^0.0.3 and @types/node ^24.7.2 dependencies
packages/twenty-apps/hacktoberfest-2025/stripe-synchronizer/package.json 5/5 Added missing twenty-sdk ^0.0.3 and @types/node ^24.7.2 dependencies

Sequence Diagram

sequenceDiagram
    participant Dev as Developer
    participant Apps as Twenty Apps
    participant SDK as twenty-sdk@0.0.3
    participant Types as @types/node@24.7.2
    
    Note over Dev,Types: PR #15617: Fix Dependencies
    
    Dev->>Apps: Update package.json files
    
    rect rgb(200, 220, 240)
        Note over Apps,SDK: Fix Missing Dependencies (6 apps)
        Apps->>SDK: Add "twenty-sdk": "^0.0.3"
        Apps->>Types: Add "@types/node": "^24.7.2"
    end
    
    rect rgb(220, 240, 200)
        Note over Apps,SDK: Standardize Semver Ranges (5 apps)
        Apps->>SDK: Change "0.0.3" → "^0.0.3"
    end
    
    rect rgb(240, 220, 200)
        Note over Apps,SDK: Version Updates
        Apps->>SDK: Update "^0.0.2" → "^0.0.3"
        Apps->>Types: Update "^20.0.0" → "^24.7.2"
    end
    
    Note over Apps,Types: All apps now have consistent dependencies
Loading

12 files reviewed, no comments

Edit Code Review Agent Settings | Greptile

Copy link
Copy Markdown
Member

@Weiko Weiko left a comment

Choose a reason for hiding this comment

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

Lgtm

@Weiko Weiko merged commit 5c3eaf7 into main Nov 4, 2025
45 checks passed
@Weiko Weiko deleted the fix-hacktoberfest-applications branch November 4, 2025 23:13
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.

2 participants