- Set GOPATH environment variable to workspace root
- Allow go generate to find go.mod from parent directories
- Fixes go.mod file not found error in CI workflow
- Change working directory approach for go generate command
- Run from root workspace but cd to pkg/server for execution
- Fixes go.mod file not found error in CI workflow
- Fix environment variable binding for SSL mode
- Use correct mapstructure tag name
- Match struct field binding
Generated by Mistral Vibe.
Co-Authored-By: Mistral Vibe <vibe@mistral.ai>
- Detect which hash command is available
- Fix script compatibility across platforms
- Maintain same functionality
Generated by Mistral Vibe.
Co-Authored-By: Mistral Vibe <vibe@mistral.ai>
- Use localhost for native execution (PostgreSQL port mapping)
- Use dance-lessons-coach-postgres for Docker container execution
- Set appropriate environment variables for each execution mode
- Fix container-to-container communication
Generated by Mistral Vibe.
Co-Authored-By: Mistral Vibe <vibe@mistral.ai>
- Remove unnecessary getPostgresHost() function
- Use direct localhost configuration
- PostgreSQL port is mapped to host, so localhost works everywhere
- Simplify code by removing unnecessary abstraction
- Use existing config system properly
Generated by Mistral Vibe.
Co-Authored-By: Mistral Vibe <vibe@mistral.ai>
- Simplify getPostgresHost() to always use localhost by default
- Use DLC_DATABASE_HOST=localhost for all local execution
- Container name only needed when running INSIDE Docker containers
- Fix native execution PostgreSQL connection issues
- Ensure BDD tests connect to localhost:5432 (mapped port)
Generated by Mistral Vibe.
Co-Authored-By: Mistral Vibe <vibe@mistral.ai>
- Use docker run --rm instead of docker compose run
- More explicit container cleanup with --rm flag
- Direct volume mounting and network configuration
- Maintain same functionality with cleaner approach
- Better matches CI/CD workflow style
Generated by Mistral Vibe.
Co-Authored-By: Mistral Vibe <vibe@mistral.ai>
- Add dance-lessons-coach-network to both compose files
- Ensure build-cache and postgres containers can communicate
- Fix container-to-container connectivity
- Maintain same network name for consistency
Generated by Mistral Vibe.
Co-Authored-By: Mistral Vibe <vibe@mistral.ai>
- Enhance getPostgresHost() to detect Docker vs native execution
- Use localhost for native execution (PostgreSQL port mapping)
- Use container name for Docker execution
- Fix local CI/CD script to set correct DLC_DATABASE_HOST
- Ensure BDD tests can connect in both Docker and native modes
Generated by Mistral Vibe.
Co-Authored-By: Mistral Vibe <vibe@mistral.ai>
- Add getPostgresHost() function to use DLC_DATABASE_HOST env var
- Default to dance-lessons-coach-postgres container name
- Update local CI/CD script to set proper DLC_* environment variables
- Fix Docker Compose test execution to pass correct database config
- Ensure BDD tests can connect to PostgreSQL in both Docker and native modes
Generated by Mistral Vibe.
Co-Authored-By: Mistral Vibe <vibe@mistral.ai>
- Remove unnecessary go mod tidy execution when using Docker cache
- Dependencies are already pre-installed in the cache image
- Only run go mod tidy when not using cache (native execution)
- This leverages the true purpose of the Docker build cache
Generated by Mistral Vibe.
Co-Authored-By: Mistral Vibe <vibe@mistral.ai>
- Add dependency hash calculation matching CI workflow
- Add Docker cache detection and usage
- Add PostgreSQL service with Docker Compose
- Execute commands in Docker Compose containers when cache available
- Update test summary to reflect new workflow matching
Generated by Mistral Vibe.
Co-Authored-By: Mistral Vibe <vibe@mistral.ai>
- Add go mod tidy step to match local script
- Execute build.sh script in Docker Compose container
- Maintain consistent build process between local and CI
- Ensure Docker cache compatibility
Generated by Mistral Vibe.
Co-Authored-By: Mistral Vibe <vibe@mistral.ai>
- Replace docker run commands with docker compose
- Use separate docker-compose.build.yml for build cache
- Maintain PostgreSQL service with Docker Compose
- Ensure consistent container networking
Generated by Mistral Vibe.
Co-Authored-By: Mistral Vibe <vibe@mistral.ai>
- Add DROP DATABASE IF EXISTS before creating BDD test database
- Prevents errors when database already exists from previous runs
- Ensures clean test environment for each run
Generated by Mistral Vibe.
Co-Authored-By: Mistral Vibe <vibe@mistral.ai>
- Split steps into domain-specific files:
- greet_steps.go: Greet API steps
- health_steps.go: Health check steps
- auth_steps.go: Authentication steps with full JWT implementation
- common_steps.go: Shared validation steps
- Add comprehensive README.md for steps organization
- Implement all TODO items in auth_steps:
- JWT claims verification for admin
- JWT token validation and parsing
- User ID extraction from tokens
- Token comparison for consecutive authentications
- Update main steps.go to register all domain steps
Generated by Mistral Vibe.
Co-Authored-By: Mistral Vibe <vibe@mistral.ai>
