Conversation
- Remove COPY scripts/ line from Dockerfile (directory was removed in 9bb242f) - Update pnpm version in Dockerfile from 10.20.0 to 10.28.2 - Remove unused github.com/gofiber/fiber/v3 dependency - Clean up indirect dependencies via go mod tidy
WalkthroughThis PR performs dependency cleanup and Docker image optimization. The main focus is removing the unused Fiber v3 web framework dependency while retaining Fiber v2, which eliminates several transitive dependencies no longer needed by the project. The Go dependency files (go.mod and go.sum) are cleaned up to remove outdated entries and reduce the dependency footprint. Additionally, the Dockerfile is updated to use a newer version of pnpm (10.28.2) and removes the scripts/ directory from the Docker image, streamlining the build process and reducing image size. These changes represent routine maintenance to keep dependencies current and remove unused code. Changes
Sequence DiagramThis diagram shows the interactions between components: sequenceDiagram
participant Docker as Docker Build
participant npm as npm
participant pnpm as pnpm v10.28.2
participant FS as File System
participant Build as Frontend Build
Note over Docker: Dockerfile Build Process
Docker->>npm: Install pnpm globally
npm->>pnpm: Install pnpm@10.28.2
Note right of pnpm: Updated from v10.20.0
Docker->>FS: Copy package.json & pnpm-lock.yaml
Note over FS: Layer caching optimization
Docker->>pnpm: Install dependencies
pnpm->>FS: Download & install packages
Docker->>FS: Copy config files
Note over FS: postcss.config.js<br/>tailwind.config.js<br/>tsconfig.json
Note over Docker,FS: scripts/ directory no longer copied
Docker->>FS: Copy internal/web/
Docker->>Build: Execute pnpm build:assets
Build->>pnpm: Run build command
pnpm->>FS: Read source files
pnpm->>FS: Generate build artifacts
FS-->>Build: Build output
Build-->>Docker: Build complete
🔗 Cross-Repository Impact AnalysisEnable automatic detection of breaking changes across your dependent repositories. → Set up now Learn more about Cross-Repository AnalysisWhat It Does
How to Enable
Benefits
Note for WindsurfPlease change the default marketplace provider to the following in the windsurf settings:Marketplace Extension Gallery Service URL: https://marketplace.visualstudio.com/_apis/public/gallery Marketplace Gallery Item URL: https://marketplace.visualstudio.com/items Entelligence.ai can learn from your feedback. Simply add 👍 / 👎 emojis to teach it your preferences. More shortcuts belowEmoji Descriptions:
Interact with the Bot:
Also you can trigger various commands with the bot by doing The current supported commands are
More commands to be added soon. |
There was a problem hiding this comment.
Pull request overview
This PR cleans up Docker configuration and Go dependencies by removing obsolete references and updating versions to match the project's current state.
Changes:
- Updated pnpm version in Dockerfile from 10.20.0 to 10.28.2 to align with package.json
- Removed obsolete
COPY scripts/directive from Dockerfile (directory no longer exists) - Removed unused
github.com/gofiber/fiber/v3direct dependency from go.mod - Cleaned up indirect dependencies (philhofer/fwd, savsgio/dictpool, savsgio/gotils, tinylib/msgp) via go mod tidy
Reviewed changes
Copilot reviewed 2 out of 3 changed files in this pull request and generated no comments.
| File | Description |
|---|---|
| Dockerfile | Updated pnpm version to 10.28.2 and removed obsolete scripts directory copy |
| go.mod | Removed unused fiber/v3 dependency and cleaned up 4 indirect dependencies |
| go.sum | Removed checksums for deleted dependencies and outdated versions |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## main #435 +/- ##
=======================================
Coverage 45.58% 45.58%
=======================================
Files 14 14
Lines 702 702
=======================================
Hits 320 320
Misses 365 365
Partials 17 17
Flags with carried forward coverage won't be shown. Click here to find out more. Continue to review full report in Codecov by Sentry.
🚀 New features to boost your workflow:
|
|
Released in v1.2.0 |
Summary
COPY scripts/line from Dockerfile (directory was removed in 9bb242f)github.com/gofiber/fiber/v3dependencygo mod tidyTest plan