Skip to content

Conversation

@Che-Zhu
Copy link
Collaborator

@Che-Zhu Che-Zhu commented Jan 9, 2026

Summary

This PR improves build reproducibility for the sandbox Docker image and adds project design documentation.

Changes

🐳 Dockerfile Improvements

Pin Next.js version with centralized configuration

  • Added NEXT_VERSION=15.5.9 environment variable for centralized version control
  • Updated both global npm install and npx command to reference ${NEXT_VERSION}
  • Ensures consistent Next.js version across the image

Why this matters:

  • Previously, create-next-app@latest was used, which could lead to inconsistent builds
  • Now the version is locked and easy to update in one place

📚 Documentation

  • Added docs/design_specification.md with project design guidelines

Testing

  • Docker build succeeds with new version pinning
  • Next.js project created in container uses expected version (15.5.9)

- Define VS Code-inspired dark theme color system
- Document two-section sidebar architecture
- Specify component standards (Button, Card, Tabs)
- Include typography, border radius, and shadow systems
- Reference related files (globals.css, style.md)
…iable

- Add NEXT_VERSION=15.5.9 environment variable for centralized version control
- Update global npm install to use create-next-app@${NEXT_VERSION}
- Update npx command to use create-next-app@${NEXT_VERSION}

This ensures version consistency and makes future upgrades easier by
requiring changes in only one place.
@github-actions
Copy link

github-actions bot commented Jan 9, 2026

✅ PR Check Results: Passed

Build Checks

Check Status
Lint & Build ✅ Passed
Docker Build ✅ Passed

✨ Great work!

All checks passed successfully. Your PR is ready for review.

Details:

  • ✅ Code quality verified (linting passed)
  • ✅ Build successful
  • ✅ Docker image build verified (linux/amd64)
    Commit: d1123d52e10d57a2e66901d17ff4f4a8e79cb76f
    Branch: regular-update

🔗 View Details:

@Che-Zhu Che-Zhu merged commit 2293bdf into main Jan 9, 2026
7 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.

2 participants