Skip to content

Conversation

@LeCyreaxYT
Copy link
Contributor

Release 0.0.5

Alec Schitzkat added 3 commits December 3, 2025 16:44
Updates environment variable configuration for greater
flexibility across development and production environments.

- Uses `BASE_URL` for both API and Site URL in production by default.
- Allows separate `API_URL` and `SITE_URL` overrides for development.
- Renames `NUXT_PUBLIC_INDEXABLE` to `PUBLIC_INDEXABLE`.
- Splits Dockerfile into backend and frontend specific files.
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This PR implements Release 0.0.5, which represents a significant architectural change to separate the previously monolithic Docker container into separate frontend and backend containers. This separation enables independent scaling, easier deployment flexibility, and clearer separation of concerns.

Key changes include:

  • Separated Docker infrastructure with independent frontend and backend Dockerfiles
  • Updated runtime configuration to use explicit BASE_URL and API_URL environment variables
  • Consolidated three Vue layouts (auth, public, default) into a single dynamic layout with route-based switching
  • Added new setup middleware to handle setup flow redirection
  • Changed plugin initialization from non-blocking to blocking to ensure stores are ready before navigation

Reviewed changes

Copilot reviewed 25 out of 26 changed files in this pull request and generated 12 comments.

Show a summary per file
File Description
frontend/package.json Version bump from 0.0.1 to 0.0.5
frontend/nuxt.config.ts Updated runtime config with separate baseUrl/apiUrl, disabled sourcemaps, removed process import
frontend/app/plugins/00.pina.client.ts Changed to async plugin with blocking store initialization
frontend/app/pages/setup.vue Removed explicit layout metadata (now determined dynamically)
frontend/app/pages/register.vue Removed explicit layout metadata
frontend/app/pages/login.vue Removed explicit layout metadata
frontend/app/pages/index.vue Removed explicit layout metadata, removed replace:true from navigation
frontend/app/pages/f/[id].vue Removed explicit layout metadata
frontend/app/middleware/01.auth.global.ts Removed replace:true from navigation calls
frontend/app/middleware/00.setup.global.ts New middleware to handle setup flow redirection
frontend/app/layouts/public.vue Deleted - functionality merged into default layout
frontend/app/layouts/default.vue Consolidated all three layouts with dynamic route-based switching
frontend/app/layouts/auth.vue Deleted - functionality merged into default layout
frontend/app/composables/useApi.ts Added type assertions for apiUrl
frontend/app/components/FormFields/FileUploadField.vue Added type assertion for apiUrl
frontend/app/app.vue Added theme initialization, removed NuxtPwaManifest, added page transition styles
frontend/Dockerfile New standalone frontend Dockerfile with Nuxt build
docker/start.sh Deleted - no longer needed with separate containers
docker/nginx.conf Deleted - no longer needed with separate containers
backend/internal/config/config.go Added ApiURL field, updated BaseURL semantics to frontend URL
backend/cmd/server/main.go Updated to use ApiURL for storage URL construction
backend/Dockerfile New standalone backend Dockerfile
README.md Updated documentation for separate container deployment
Dockerfile Deleted monolithic Dockerfile
.github/workflows/release.yml Split into separate backend and frontend build jobs
.env.example Updated configuration structure for separate services

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

LeCyreaxYT and others added 9 commits December 3, 2025 19:33
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
@LeCyreaxYT LeCyreaxYT merged commit b75944d into main Dec 3, 2025
3 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