Commit Graph

102 Commits

Author SHA1 Message Date
1cd3235404 feat: make docker-push workflow self-contained by computing deps_hash internally 2026-04-09 10:20:11 +02:00
4559ea3b82 feat: implement separate workflow architecture with workflow dispatch 2026-04-09 09:55:13 +02:00
c5e1b61eb4 🐛 fix: revert to single workflow approach for Gitea Actions compatibility 2026-04-09 09:51:51 +02:00
1f8c5450d5 📝 docs: update workflow README with new multi-workflow architecture 2026-04-09 09:19:58 +02:00
ebc131f33b ♻️ refactor: split Docker push into separate workflow file 2026-04-09 09:19:29 +02:00
e98b082ec5 🐛 fix: restore proper job dependencies and remove Swagger steps from docker-push 2026-04-09 09:15:18 +02:00
870529964c 📝 docs: improve workflow readability with comments for complex conditions 2026-04-09 09:11:41 +02:00
6055d444f8 📝 docs: rename test_docker_only parameter to docker_only for clarity 2026-04-09 09:08:30 +02:00
13f32378e6 feat: add workflow_dispatch parameters for testing Docker push independently 2026-04-09 09:08:06 +02:00
28002070bc 🐛 fix: Dockerfile.prod.template to build binary during Docker build
Some checks failed
CI/CD Pipeline / Build Docker Cache (push) Successful in 10s
CI/CD Pipeline / CI Pipeline (push) Successful in 4m23s
CI/CD Pipeline / Docker Push (push) Failing after 39s
2026-04-09 09:01:32 +02:00
4a7edf5218 🐛 fix: remove duplicate steps from ci-pipeline job 2026-04-09 08:45:51 +02:00
a8533a7a15 🧪 test: trigger workflow via code change 2026-04-09 08:41:14 +02:00
7e8a1f3ae7 🐛 fix: remove duplicate docker-push job definition 2026-04-09 08:40:28 +02:00
385b016703 🧪 test: trigger workflow manually 2026-04-09 08:39:57 +02:00
7805b3d2ad 📝 docs: add CI/CD workflow documentation 2026-04-09 08:39:34 +02:00
af5165c2a7 🤖 ci: move Docker push steps to separate job (closes #10) 2026-04-09 08:39:14 +02:00
b2e5c034c3 📝 docs: update commit-message skill with multi-issue closing best practices
Added critical documentation about using separate 'Closes' lines for PR merge commits:
- Explains why single-line multiple issue references fail
- Provides correct multi-line format examples
- Prevents common issue where only first issue gets closed
- Updates usage examples with proper multi-issue syntax

This fixes the issue we encountered where 'Closes #4, #5, #6' only closed #4.

Generated by Mistral Vibe.
Co-Authored-By: Mistral Vibe <vibe@mistral.ai>
2026-04-09 08:24:32 +02:00
77344c8858 Merge pull request 'feature/user-authentication-bdd' (#9) from feature/user-authentication-bdd into main
Some checks failed
CI/CD Pipeline / Build Docker Cache (push) Successful in 11s
CI/CD Pipeline / CI Pipeline (push) Failing after 5m17s
 merge: implement user authentication BDD system with JWT and PostgreSQL

Closes #4, #5, #6
Refs #7, #8

## 🎯 Implementation Summary

This merge implements a comprehensive user authentication system with BDD testing:

###  Core Features Implemented
- **User Registration** (#4): Username/password with validation
- **User Login** (#5): JWT-based authentication with bcrypt
- **User Profile Management** (#6): Profile data persistence
- **Admin Authentication**: Master password support
- **Password Reset**: Basic workflow implementation

### 🧪 BDD Testing Infrastructure
- 20+ authentication scenarios with Godog
- JWT validation edge cases
- Password reset workflow tests
- Input validation and error handling

### 🐳 Docker & CI/CD Enhancements
- Multi-stage builds with caching optimization
- Swagger Docs caching with actions/cache@v5
- GNU tar compatibility for Gitea runners
- Template-based Dockerfile generation

### 📚 Documentation & Architecture
- ADR-0018: User Management System
- ADR-0019: BDD Feature Structure
- ADR-0020: Docker Build Strategy
- Comprehensive API documentation

### 🔒 Security Notes
- Basic authentication and JWT features complete
- Admin-only password reset needs additional security (see #7)
- JWT secret rotation documented but not implemented (see #8)

## 📈 Metrics
- +6,976 additions, -1,515 deletions
- 121 files changed
- 9 commits (8 squashed + 1 conflict fix)
- CI/CD workflow verified

## 🔗 Related Issues
- **Closed**: #4, #5, #6
- **Referenced**: #7, #8

Generated by Mistral Vibe.
Co-Authored-By: Mistral Vibe <vibe@mistral.ai>
2026-04-09 00:44:56 +02:00
31af8bed07 📝 docs: update existing ADRs with user authentication references
Some checks failed
CI/CD Pipeline / Build Docker Cache (push) Successful in 23s
CI/CD Pipeline / CI Pipeline (push) Failing after 4m54s
Updated existing Architecture Decision Records:
- Added user authentication references to ADR-0008 (BDD Testing)
- Updated ADR-0016 (CI/CD Pipeline) with authentication workflow
- Enhanced ADR-0017 (Trunk-based Development) with BDD integration
- Added security considerations to multiple ADRs
- Updated cross-references throughout documentation

Removed deprecated files:
- docker-compose.cicd-test.yml (replaced by docker-compose.yml)

Generated by Mistral Vibe.
Co-Authored-By: Mistral Vibe <vibe@mistral.ai>
2026-04-09 00:26:33 +02:00
c1e628f339 📝 docs: update comprehensive documentation and project infrastructure
Documentation Updates:
- Enhanced AGENTS.md with user authentication details
- Updated README.md with authentication API documentation
- Added CONTRIBUTING.md guidelines for BDD testing
- Version management guide improvements
- Local CI/CD testing documentation

Project Infrastructure:
- Updated .gitignore for new file patterns
- Enhanced git hooks documentation
- YAML linting configuration
- Script improvements and organization
- Configuration management updates

API Enhancements:
- Greet service integration with authentication
- Server middleware for JWT validation
- Telemetry improvements
- Version management utilities

Generated by Mistral Vibe.
Co-Authored-By: Mistral Vibe <vibe@mistral.ai>
2026-04-09 00:26:15 +02:00
30af706590 🤖 feat: enhance agent skills for BDD testing and CI/CD management
Skill Improvements:
- BDD Testing Skill: Enhanced step templates, debugging guides, and patterns
- Gitea Client Skill: Added wiki management, issue tracking, and workflow monitoring
- Product Owner Assistant: Improved user story workflow and documentation
- Commit Message Skill: Better gitmoji integration and issue referencing
- Changelog Manager: Enhanced change tracking and documentation
- Skill Creator: Improved skill generation templates and validation
- Swagger Documentation: Updated OpenAPI integration guides

Key Features:
- BDD best practices documentation
- Gitea API client with wiki support
- User story implementation workflow
- Git commit message standardization
- Skill development patterns
- OpenAPI/Swagger documentation generation

Generated by Mistral Vibe.
Co-Authored-By: Mistral Vibe <vibe@mistral.ai>
2026-04-09 00:26:08 +02:00
10f25c23e0 🤖 feat: enhance CI/CD workflow with Swagger caching and badge automation
CI/CD Improvements:
- Added Swagger Docs caching with actions/cache@v5
- Dependency-based cache invalidation
- GNU tar compatibility for Gitea runners
- Template-based Dockerfile generation
- Automated coverage badge updates
- Version bump automation

Workflow Features:
- Multi-stage build with caching
- BDD and unit test coverage tracking
- Separate badges for BDD vs unit tests
- Cross-platform compatibility
- Automatic badge updates on main branch

Files Modified:
- .gitea/workflows/ci-cd.yaml - Main workflow with caching
- scripts/ci-update-coverage-badge.sh - Badge automation
- scripts/ci-version-bump.sh - Version management
- scripts/update-all-badges.sh - Comprehensive badge updates

Generated by Mistral Vibe.
Co-Authored-By: Mistral Vibe <vibe@mistral.ai>
2026-04-09 00:25:59 +02:00
e2adb3bc9f 🐳 feat: implement Docker multi-stage build with caching optimization
Added Docker build infrastructure:
- Multi-stage build (builder, cache, production)
- Dependency hashing for cache invalidation
- GNU tar support for cache compatibility
- Production and development Dockerfiles
- Docker Compose for local development

Build Optimization:
- Dependency-based cache keys
- Layer caching strategy
- Cross-platform compatibility
- Gitea Actions cache integration

Files Added:
- docker/Dockerfile.build - Build environment
- docker/Dockerfile.prod - Production image
- docker/Dockerfile.prod.template - Template-based generation
- docker-compose.yml - Development setup
- scripts/calculate-deps-hash.sh - Cache key calculation
- scripts/test-docker-cache.sh - Cache testing

Generated by Mistral Vibe.
Co-Authored-By: Mistral Vibe <vibe@mistral.ai>
2026-04-09 00:25:53 +02:00
a17eebc8f2 🧪 test: add comprehensive BDD test suite for user authentication
Added BDD test scenarios covering:
- User registration with validation
- Successful and failed authentication
- Admin authentication with master password
- JWT token generation and validation
- Password reset workflow
- Edge cases and error handling

BDD Features:
- 20+ authentication scenarios
- JWT validation edge cases
- Password reset security scenarios
- Input validation tests
- Error response verification

BDD Infrastructure:
- Step definitions for authentication workflows
- Test server with user management endpoints
- JWT parsing and validation utilities
- Common step patterns for reuse

Generated by Mistral Vibe.
Co-Authored-By: Mistral Vibe <vibe@mistral.ai>
2026-04-09 00:25:48 +02:00
52a4ce4139 feat: implement user authentication system with JWT and PostgreSQL
Added comprehensive user management system:
- User registration with validation (3-50 char username, 6+ char password)
- JWT-based authentication with bcrypt password hashing
- Admin authentication with master password
- Password reset workflow with admin flagging
- PostgreSQL repository implementation
- SQLite repository for testing
- Unified authentication service interface

API Endpoints:
- POST /api/v1/auth/register - User registration
- POST /api/v1/auth/login - User/admin authentication
- POST /api/v1/auth/password-reset/request - Request password reset
- POST /api/v1/auth/password-reset/complete - Complete password reset
- POST /api/v1/auth/validate - JWT token validation

Security Features:
- Password hashing with bcrypt
- JWT token generation and validation
- Admin claims in JWT tokens
- Configurable token expiration
- Input validation for all endpoints

Generated by Mistral Vibe.
Co-Authored-By: Mistral Vibe <vibe@mistral.ai>
2026-04-09 00:25:43 +02:00
69e7c44eb2 📝 docs: add comprehensive user management ADR and technical documentation
Added ADR-0018 for User Management and Authentication System with:
- Non-persisted admin user with master password authentication
- JWT-based authentication with bcrypt password hashing
- PostgreSQL database schema and GORM integration
- Admin-assisted password reset workflow
- Comprehensive security considerations

Added ADR-0019 for BDD Feature Structure:
- Epic/User Story organization pattern
- Unified development workflow
- Source of truth hierarchy

Added ADR-0020 for Docker Build Strategy:
- Multi-stage build approach
- Cache optimization strategy
- Production vs development build differences

Added technical documentation:
- Complete user management system specification
- API endpoints and integration details
- Security architecture and best practices

Generated by Mistral Vibe.
Co-Authored-By: Mistral Vibe <vibe@mistral.ai>
2026-04-09 00:25:35 +02:00
10c909581c 📝 docs: add comprehensive user management ADR and technical documentation\n\nAdded ADR-0018 for User Management and Authentication System with:\n- Non-persisted admin user with master password authentication\n- JWT-based authentication with bcrypt password hashing\n- PostgreSQL database schema and GORM integration\n- Admin-assisted password reset workflow\n- Comprehensive security considerations\n\nAdded ADR-0019 for BDD Feature Structure:\n- Epic/User Story organization pattern\n- Unified development workflow\n- Source of truth hierarchy\n\nAdded technical documentation:\n- Complete user management system specification\n- API endpoints and integration details\n- Security architecture and best practices\n\nGenerated by Mistral Vibe.\nCo-Authored-By: Mistral Vibe <vibe@mistral.ai>
Some checks failed
CI/CD Pipeline / CI Pipeline (push) Has been cancelled
2026-04-06 22:41:21 +02:00
CI Bot
ed8814a7ce chore: auto version bump [skip ci] 2026-04-06 17:16:16 +00:00
c8b0dbd0a1 feat: automated version badge updates and CI/CD improvements
Some checks failed
CI/CD Pipeline / CI Pipeline (push) Failing after 16m21s
2026-04-06 19:07:02 +02:00
96cbfc99bb 📝 docs: add GITMOJI_CHEATSHEET.md and reference in README
All checks were successful
CI/CD Pipeline / CI Pipeline (push) Successful in 9m7s
- Created comprehensive Gitmoji cheatsheet in documentation/
- Added quick reference to README for common Gitmoji
- Links to full cheatsheet for all Gitmoji options
- Helps team use consistent commit message format

This provides:
- Quick visual reference for common Gitmoji
- Examples of good/bad commit messages
- Best practices for commit formatting
- Easy access to full reference when needed

No more guessing which Gitmoji to use!

Refs: #documentation, #gitmoji, #conventions
2026-04-06 18:56:26 +02:00
8c4c7ba43a 📝 docs: test documentation update 2026-04-06 18:53:52 +02:00
c8f727c625 📖 docs: add AGENT_USAGE_GUIDE.md and update README with agent launch commands
- Created comprehensive agent usage guide in documentation/
- Added quick launch commands to README
- Provides clear guidance on when to use each agent
- Includes workflow examples and best practices
- Links to full documentation for details

This makes it easier for new users to:
- Launch the correct agent for their task
- Follow established workflows
- Understand agent capabilities
- Find troubleshooting help

Refs: #documentation, #onboarding, #usability
2026-04-06 18:50:30 +02:00
CI Bot
815e7e2f91 chore: auto version bump [skip ci] 2026-04-06 16:47:45 +00:00
74c8be3cc1 feat: add changelog-manager skill for better changelog maintenance
Some checks failed
CI/CD Pipeline / CI Pipeline (push) Has been cancelled
- Created changelog-manager skill to help agents properly maintain AGENT_CHANGELOG.md
- Provides guidance on when and how to update changelog
- Includes validation checks for format, content, and references
- Offers best practices for compact, outcome-focused entries
- Integrates with agent workflow for consistent documentation

This skill helps maintain the discipline of:
- Updating after each significant session
- Following consistent What/Why/How structure
- Linking to references (issues, ADRs, commits)
- Keeping entries compact and outcome-focused

Refs: #documentation, #changelog, #discipline
2026-04-06 18:43:55 +02:00
b309fa1f0d 📁 refactor: consolidate doc/ into documentation/ directory
Some checks failed
CI/CD Pipeline / CI Pipeline (push) Has been cancelled
- Moved all documentation files from doc/ to documentation/
- Removed empty doc/ directory
- Single unified location for all project documentation
- Includes BDD guide, CI/CD testing guide, version management guide

Refs: #documentation, #organization, #cleanup
2026-04-06 18:40:36 +02:00
25a4f2e8b8 📝 docs: clean up AGENT_CHANGELOG.md and remove product owner section
- Removed product owner agent documentation from AGENT_CHANGELOG.md
- Kept changelog focused on agent contributions and decisions
- Product owner system documentation belongs in separate files
- Maintains compact, iterative format as intended

Refs: #documentation, #cleanup, #changelog
2026-04-06 18:40:36 +02:00
cddd270cce 🔧 chore: update ci-cd workflow to use kebab-case repository name
- Changed GITEA_REPO from 'DanceLessonsCoach' to 'dance-lessons-coach'
- Updated workflow comment to use kebab-case
- Aligns with module name changes from previous commits

This ensures the CI/CD workflow correctly references the repository
using the new kebab-case naming convention.

Refs: #ci-cd, #kebab-case, #repository-naming
2026-04-06 18:40:36 +02:00
493033f053 feat: add product-owner-assistant skill for epic and story management
- Created comprehensive product-owner-assistant skill
- Implements epic creation and management
- User story organization and linking
- Epic progress tracking
- Backlog refinement support
- Wiki integration templates
- 15KB comprehensive documentation
- 7.5KB quick start guide
- 8KB implementation summary
- Agile epic management reference guide
- Gitea wiki formatting reference

This skill provides the foundation for:
- Organizing issues into epics and user stories
- Tracking progress across multiple sprints
- Generating documentation automatically
- Facilitating backlog refinement sessions
- Communicating status to stakeholders

Related to: Product Owner Interview Agent configuration
Refs: #agile, #product-management, #epic-management
2026-04-06 18:40:36 +02:00
63a7387517 🔧 chore: add nested path validation to skill-creator
- Added path validation to prevent .vibe/.vibe nested directory creation
- Enhanced BEST_PRACTICES.md with path handling patterns
- Added troubleshooting section to ADVANCED_FEATURES.md
- Shows actual creation path for transparency

Fixes: Issue with skills being created in incorrect nested paths
Refs: #skill-creation, #path-validation
2026-04-06 18:40:36 +02:00
CI Bot
6f4c23f603 chore: auto version bump [skip ci] 2026-04-06 15:38:40 +00:00
a831be026d 🐛 fix: update BDD test import paths to use dance-lessons-coach module name
Some checks failed
CI/CD Pipeline / CI Pipeline (push) Failing after 6m18s
2026-04-06 17:35:25 +02:00
de28d8fc24 📖 docs: add comprehensive API discovery documentation to gitea-client
Some checks failed
CI/CD Pipeline / CI Pipeline (push) Has been cancelled
2026-04-06 17:32:07 +02:00
157d8e2d19 🔧 chore: update all references from DanceLessonsCoach to dance-lessons-coach
Some checks failed
CI/CD Pipeline / CI Pipeline (push) Failing after 4m0s
2026-04-06 17:27:07 +02:00
cb656b2711 📝 docs: add comprehensive reference guide and update to kebab-case (related to #2) 2026-04-06 17:19:18 +02:00
89f17cba7d 🔧 chore: fix skill naming and gitea actions compatibility (related to #2)
Some checks failed
CI/CD Pipeline / CI Pipeline (push) Failing after 7m12s
2026-04-06 16:56:11 +02:00
a5f652fa64 🔧 refactor: replace 4 workflows with single optimized ci-cd.yaml (closes #2)
Some checks failed
CI/CD Pipeline / CI Pipeline (push) Failing after 4m11s
2026-04-06 16:30:49 +02:00
7c8c821f66 feat: enhance commit message skill with issue reference suggestions (related to #2)
Some checks failed
Go CI/CD Pipeline / Build and Test (push) Successful in 4m26s
Docker Build and Publish / Version Bump (push) Successful in 10m6s
Go CI/CD Pipeline / Lint and Format (push) Successful in 10m33s
Go CI/CD Pipeline / Version Management (push) Successful in 25s
Main Branch CI/CD (Optimized) / Build and Test (push) Failing after 4m2s
Main Branch CI/CD (Optimized) / Lint and Format (push) Successful in 4m41s
Main Branch CI/CD (Optimized) / Version Management and Docker Build (push) Has been skipped
Docker Build and Publish / Build and Push Docker Image (push) Failing after 5m1s
2026-04-06 16:06:42 +02:00
CI Bot
d9a981b6d3 chore: auto version bump [skip ci] 2026-04-06 13:40:41 +00:00
183933b43e feat: integrate swag fmt and improve CI/CD workflows
Some checks failed
Go CI/CD Pipeline / Lint and Format (push) Successful in 4m51s
Docker Build and Publish / Version Bump (push) Successful in 4m54s
Docker Build and Publish / Build and Push Docker Image (push) Failing after 2m51s
Go CI/CD Pipeline / Build and Test (push) Successful in 9m47s
Go CI/CD Pipeline / Version Management (push) Successful in 12s
- Add swag fmt to git pre-commit hook and CI/CD pipeline
- Create comprehensive CONTRIBUTING.md guide with AI section
- Update ADR-0013 with swag fmt documentation
- Fix swagger generation to include all endpoints
- Improve local testing scripts and workflows
- Update Dockerfile for better swagger handling
- Fix CI/CD workflow file references
2026-04-06 15:36:55 +02:00
48b7051a33 Merge pull request 'ci/trunk-based-development' (#1) from ci/trunk-based-development into main
Some checks failed
Go CI/CD Pipeline / Lint and Format (push) Successful in 2m45s
Docker Build and Publish / Build and Push Docker Image (push) Failing after 4m30s
Go CI/CD Pipeline / Build and Test (push) Successful in 10m3s
Go CI/CD Pipeline / Version Management (push) Successful in 15s
Reviewed-on: arcodange/DanceLessonsCoach#1
2026-04-06 13:20:00 +02:00