Commit Graph
100 Commits
Author SHA1 Message Date
jason.woltje d2c7602430 test(#405): add E2E integration tests for speech services
Adds comprehensive integration tests covering all 9 required scenarios:
1. REST transcription (POST /speech/transcribe)
2. REST synthesis (POST /speech/synthesize)
3. Provider fallback (premium -> default -> fallback chain)
4. WebSocket streaming transcription lifecycle
5. Audio MIME type validation (reject invalid formats)
6. File size limit enforcement (25 MB max)
7. Authentication on all endpoints (401 without token)
8. Voice listing with tier filtering (GET /speech/voices)
9. Health check status (GET /speech/health)

Uses NestJS testing module with mocked providers (CI-compatible).
30 test cases, all passing.

Fixes #405
2026-02-15 03:26:05 -06:00
jason.woltjeandClaude Opus 4.6 24065aa199 docs(#406): add speech services documentation
Comprehensive documentation for the speech services module:
- docs/SPEECH.md: Architecture, API reference, WebSocket protocol,
  environment variables, provider configuration, Docker setup,
  GPU VRAM budget, and frontend integration examples
- apps/api/src/speech/AGENTS.md: Module structure, provider pattern,
  how to add new providers, gotchas, and test patterns
- README.md: Speech capabilities section with quick start

Fixes #406

Co-Authored-By: Claude Opus 4.6 <[email protected]>
2026-02-15 03:23:22 -06:00
jason.woltjeandClaude Opus 4.6 bc86947d01 feat(#404): add speech settings page with provider config
Implements the SpeechSettings component with four sections:
- STT settings (enable/disable, language preference)
- TTS settings (enable/disable, voice selector, tier preference, auto-play, speed control)
- Voice preview with test button
- Provider status with health indicators

Also adds Slider UI component and getHealthStatus API client function.
30 unit tests covering all sections, toggles, voice loading, and PDA-friendly design.

Fixes #404

Co-Authored-By: Claude Opus 4.6 <[email protected]>
2026-02-15 03:16:27 -06:00
jason.woltjeandClaude Opus 4.6 74d6c1092e feat(#403): add audio playback component for TTS output
Implements AudioPlayer inline component with play/pause, progress bar,
speed control (0.5x-2x), download, and duration display. Adds
TextToSpeechButton "Read aloud" component that synthesizes text via
the speech API and integrates AudioPlayer for playback. Includes
useTextToSpeech hook with API integration, audio caching, and
playback state management. All 32 tests passing.

Co-Authored-By: Claude Opus 4.6 <[email protected]>
2026-02-15 03:05:39 -06:00
jason.woltjeandClaude Opus 4.6 28c9e6fe65 feat(#397): implement WebSocket streaming transcription gateway
Add SpeechGateway with Socket.IO namespace /speech for real-time
streaming transcription. Supports start-transcription, audio-chunk,
and stop-transcription events with session management, authentication,
and buffer size rate limiting. Includes 29 unit tests covering
authentication, session lifecycle, error handling, cleanup, and
client isolation.

Co-Authored-By: Claude Opus 4.6 <[email protected]>
2026-02-15 02:54:41 -06:00
jason.woltje b3d6d73348 feat(#400): add Docker Compose swarm/prod deployment for speech services
Add docker/docker-compose.sample.speech.yml for standalone speech services
deployment in Docker Swarm with Portainer compatibility:

- Speaches (STT + basic TTS) with Whisper model configuration
- Kokoro TTS (default high-quality TTS) always deployed
- Chatterbox TTS (premium, GPU) commented out as optional
- Traefik labels for reverse proxy routing with TLS
- Health checks on all services
- Volume persistence for Whisper models
- GPU reservation via Swarm generic resources for Chatterbox
- Environment variable substitution for Portainer
- Comprehensive header documentation

Fixes #400
2026-02-15 02:51:13 -06:00
jason.woltjeandClaude Opus 4.6 527262af38 feat(#392): create /api/speech/transcribe REST endpoint
Add SpeechController with POST /api/speech/transcribe for audio
transcription and GET /api/speech/health for provider status.
Uses AudioValidationPipe for file upload validation and returns
results in standard { data: T } envelope.

Includes 10 unit tests covering transcribe with options, error
propagation, and all health status combinations.

Fixes #392

Co-Authored-By: Claude Opus 4.6 <[email protected]>
2026-02-15 02:47:52 -06:00
jason.woltjeandClaude Opus 4.6 6c465566f6 feat(#395): implement Piper TTS provider via OpenedAI Speech
Add fallback-tier TTS provider using Piper via OpenedAI Speech for
ultra-lightweight CPU-only synthesis. Maps 6 standard OpenAI voice
names (alloy, echo, fable, onyx, nova, shimmer) to Piper voices.
Update factory to use the new PiperTtsProvider class, replacing the
inline stub. Includes 37 unit tests covering provider identity,
voice mapping, and voice listing.

Fixes #395

Co-Authored-By: Claude Opus 4.6 <[email protected]>
2026-02-15 02:39:20 -06:00
jason.woltje 7b4fda6011 feat(#398): add audio/text validation pipes and speech DTOs
Create AudioValidationPipe for MIME type and file size validation,
TextValidationPipe for TTS text input validation, and DTOs for
transcribe/synthesize endpoints. Includes 36 unit tests.

Fixes #398
2026-02-15 02:37:54 -06:00
jason.woltjeandClaude Opus 4.6 d37c78f503 feat(#394): implement Chatterbox TTS provider with voice cloning
Add ChatterboxSynthesizeOptions interface with referenceAudio and
emotionExaggeration fields, and comprehensive unit tests (26 tests)
covering voice cloning, emotion control, clamping, graceful degradation,
and cross-language support.

Co-Authored-By: Claude Opus 4.6 <[email protected]>
2026-02-15 02:29:38 -06:00
jason.woltjeandClaude Opus 4.6 79b1d81d27 feat(#393): implement Kokoro-FastAPI TTS provider with voice catalog
Extract KokoroTtsProvider from factory into its own module with:
- Full voice catalog of 54 built-in voices across 8 languages
- Voice metadata parsing from ID prefix (language, gender, accent)
- Exported constants for supported formats and speed range
- Comprehensive unit tests (48 tests)
- Fix lint/type errors in chatterbox provider (Prettier + unsafe cast)

Co-Authored-By: Claude Opus 4.6 <[email protected]>
2026-02-15 02:27:47 -06:00
jason.woltje b5edb4f37e feat(#391): add base TTS provider and factory classes
Add the BaseTTSProvider abstract class and TTS provider factory that were
part of the tiered TTS architecture but missed from the previous commit.

- BaseTTSProvider: abstract base with synthesize(), listVoices(), isHealthy()
- tts-provider.factory: creates Kokoro/Chatterbox/Piper providers from config
- 30 tests (22 base provider + 8 factory)

Refs #391
2026-02-15 02:20:24 -06:00
jason.woltje 3ae9e53bcc feat(#391): implement tiered TTS provider architecture with base class
Add abstract BaseTTSProvider class that implements common OpenAI-compatible
TTS logic using the OpenAI SDK with configurable baseURL. Includes synthesize(),
listVoices(), and isHealthy() methods. Create TTS provider factory that
dynamically registers Kokoro (default), Chatterbox (premium), and Piper
(fallback) providers based on configuration. Update SpeechModule to use
the factory for TTS_PROVIDERS injection token.

Also fixes lint error in speaches-stt.provider.ts (Array<T> -> T[]).

30 tests added (22 base provider + 8 factory), all passing.

Fixes #391
2026-02-15 02:19:46 -06:00
jason.woltjeandClaude Opus 4.6 c40373fa3b feat(#389): create SpeechModule with provider abstraction layer
Add SpeechModule with provider interfaces and service skeleton for
multi-tier TTS fallback (premium -> default -> fallback) and STT
transcription support. Includes 27 unit tests covering provider
selection, fallback logic, and availability checks.

- ISTTProvider interface with transcribe/isHealthy methods
- ITTSProvider interface with synthesize/listVoices/isHealthy methods
- Shared types: SpeechTier, TranscriptionResult, SynthesisResult, etc.
- SpeechService with graceful TTS fallback chain
- NestJS injection tokens (STT_PROVIDER, TTS_PROVIDERS)
- SpeechModule registered in AppModule
- ConfigModule integration via speechConfig registerAs factory

Co-Authored-By: Claude Opus 4.6 <[email protected]>
2026-02-15 02:09:45 -06:00
jason.woltjeandClaude Opus 4.6 52553c8266 feat(#399): add Docker Compose dev overlay for speech services
Add docker-compose.speech.yml with three speech services:
- Speaches (STT via Whisper + basic TTS) on port 8090
- Kokoro-FastAPI (default TTS) on port 8880
- Chatterbox TTS (premium, GPU-required) on port 8881 behind
  the premium-tts profile

All services include health checks, connect to the mosaic-internal
network, and follow existing naming/labeling conventions. Makefile
targets added: speech-up, speech-down, speech-logs.

Fixes #399

Co-Authored-By: Claude Opus 4.6 <[email protected]>
2026-02-15 02:06:21 -06:00
jason.woltje 4cc43bece6 feat(#401): add speech services config and env vars
Add SpeechConfig with typed configuration and startup validation for
STT (Whisper/Speaches), TTS default (Kokoro), TTS premium (Chatterbox),
and TTS fallback (Piper/OpenedAI). Includes registerAs factory for
NestJS ConfigModule integration, .env.example documentation, and 51
unit tests covering all validation paths.

Refs #401
2026-02-15 02:03:21 -06:00
jason.woltjeandClaude Opus 4.6 fb53272fa9 chore(orchestrator): Bootstrap M13-SpeechServices tasks.md
18 tasks across 7 phases for TTS & STT integration.
Estimated total: ~322K tokens.

Co-Authored-By: Claude Opus 4.6 <[email protected]>
2026-02-15 01:56:06 -06:00
jason.woltjeandClaude Opus 4.6 8ce6843af2 fix(database,api): add 6 missing table migrations and fix CORS health checks
Database: 6 models in the Prisma schema had no CREATE TABLE migration:
cron_schedules, workspace_llm_settings, quality_gates, task_rejections,
token_budgets, llm_usage_logs. Same root cause as the federation tables.

CORS: Health check requests (Docker, load balancers) don't send Origin
headers. The CORS config was rejecting these in production, causing
/health to return 500 and Docker to mark the container as unhealthy.
Requests without Origin headers are not cross-origin per the CORS spec
and should be allowed through.

Co-Authored-By: Claude Opus 4.6 <[email protected]>
2026-02-15 01:49:13 -06:00
jason.woltjeandClaude Opus 4.6 dfe89b7a3b fix(devops): add CSRF_SECRET to all compose files
Added CSRF_SECRET to docker-compose.swarm.portainer.yml (the active
Portainer deployment) and both example compose files. Also added
ENCRYPTION_KEY to the example files where it was missing.

Co-Authored-By: Claude Opus 4.6 <[email protected]>
2026-02-15 01:44:45 -06:00
jason.woltjeandClaude Opus 4.6 7aee5ed5ba fix(devops): add CSRF_SECRET and ENCRYPTION_KEY to compose files
Both env vars were missing from the API service environment in
docker-compose.prod.yml and docker-compose.build.yml, causing the
CSRF_SECRET check to fail at startup even when set in .env.

Co-Authored-By: Claude Opus 4.6 <[email protected]>
2026-02-15 01:41:35 -06:00
jason.woltjeandClaude Opus 4.6 3d54f7a7f0 docs: add CSRF_SECRET to .env.example
Co-Authored-By: Claude Opus 4.6 <[email protected]>
2026-02-15 01:36:55 -06:00
jason.woltjeandClaude Opus 4.6 6e20fc5d16 feat: Sample Matrix swarm deployment compose file (#387)
Standalone Synapse + Element Web deployment for Docker Swarm/Portainer.
Separate infrastructure from Mosaic Stack (same pattern as Authentik).

Includes: Synapse, Element Web, dedicated PostgreSQL, optional coturn.
Traefik labels match existing Stack conventions.

Refs #387

Co-Authored-By: Claude Opus 4.6 <[email protected]>
2026-02-15 01:12:41 -06:00
jason.woltjeandClaude Opus 4.6 d2003a7b03 fix(api): make federation config validation non-fatal at startup
Federation is optional and should not prevent the app from starting
when DEFAULT_WORKSPACE_ID is not set. Changed from throwing (crash)
to logging a warning. The endpoint-level validation in the controller
still rejects requests when federation is unconfigured.

Co-Authored-By: Claude Opus 4.6 <[email protected]>
2026-02-15 01:08:09 -06:00
jason.woltjeandClaude Opus 4.6 8733a643bf fix(api): remove "type": "module" conflicting with CommonJS build output
The NestJS tsconfig compiles to CommonJS (module: "CommonJS") but
package.json had "type": "module", causing Node.js v24 to treat the
CJS output as ESM and fail with "exports is not defined in ES module
scope" at startup.

Co-Authored-By: Claude Opus 4.6 <[email protected]>
2026-02-15 00:53:43 -06:00
jason.woltjeandClaude Opus 4.6 91307c87cc fix(database): add missing federation table migrations
Federation models (FederationConnection, FederatedIdentity,
FederationMessage) and their enums were defined in the Prisma schema
but never had CREATE TABLE migrations. This caused the
20260203_add_federation_event_subscriptions migration to fail with
"relation federation_messages does not exist".

Adds new migration 20260202200000 to create the 3 missing enums,
3 missing tables, all indexes, and foreign keys. Removes the
now-redundant ALTER TABLE from the 20260203 migration since
event_type is created with the table.

Co-Authored-By: Claude Opus 4.6 <[email protected]>
2026-02-15 00:29:37 -06:00
jason.woltjeandClaude Opus 4.6 f4e759c07a fix(devops): bypass OpenBao base entrypoint to prevent dev-mode flags
The base openbao image's docker-entrypoint.sh injects -dev-root-token-id
and -dev-listen-address flags when it sees 'server' as $1, causing the
server to exit immediately (code 0). Override entrypoint with dumb-init
and call bao directly to avoid the dev-mode flag injection.

Co-Authored-By: Claude Opus 4.6 <[email protected]>
2026-02-15 00:13:57 -06:00
jason.woltjeandClaude Opus 4.6 b6d272992a fix(devops): fix OpenBao healthcheck URL truncation with CMD-SHELL
The CMD exec form drops everything after & in the healthcheck URL,
causing uninitcode=200 and sealedcode=200 params to be lost. Without
them, OpenBao returns 501 when uninitialized, healthcheck fails, and
Swarm kills the container before the init sidecar can reach it.

Switch to CMD-SHELL with single-quoted URL to preserve query params.

Co-Authored-By: Claude Opus 4.6 <[email protected]>
2026-02-15 00:08:12 -06:00
jason.woltjeandClaude Opus 4.6 14162b9213 fix(api): use node_modules prisma binary in entrypoint
npx is unavailable in production image since npm is removed.
Use ./node_modules/.bin/prisma directly instead.

Co-Authored-By: Claude Opus 4.6 <[email protected]>
2026-02-15 00:05:46 -06:00
jason.woltjeandClaude Opus 4.6 44a44b5f56 fix(ci): remove SHA tags, use only dev/latest/vX.X.X
Align image tagging with semver convention:
- develop branch → :dev
- main branch → :latest
- git tags → :vX.X.X

Removes commit SHA tags from all 5 pipelines (api, web, orchestrator,
coordinator, infra) and updates Trivy scans to reference branch/tag.

Co-Authored-By: Claude Opus 4.6 <[email protected]>
2026-02-14 23:58:51 -06:00
jason.woltjeandClaude Opus 4.6 899faba7e2 fix(devops): set Valkey maxmemory-policy to noeviction for BullMQ
BullMQ requires noeviction to prevent silent job data loss. With
allkeys-lru, Valkey could evict keys BullMQ depends on for job tracking.

Co-Authored-By: Claude Opus 4.6 <[email protected]>
2026-02-14 16:51:42 -06:00
jason.woltjeandClaude Opus 4.6 bcee4fa601 fix(api): auto-run migrations on container start and fix ESM warning
- Add docker-entrypoint.sh that runs prisma migrate deploy before
  starting the app, ensuring all tables exist on deployment
- Add "type": "module" to package.json to eliminate Node.js ESM
  reparsing warning for eslint.config.js

Co-Authored-By: Claude Opus 4.6 <[email protected]>
2026-02-14 16:47:57 -06:00
jason.woltjeandClaude Opus 4.6 ab52827d9c chore: add install scripts, doctor command, and AGENTS.md
- Add one-line installer (scripts/install.sh) with platform detection
- Add doctor command (scripts/commands/doctor.sh) for environment diagnostics
- Add shared libraries: dependencies, docker, platform, validation
- Update README with quick-start installer instructions
- Add AGENTS.md with codebase patterns for AI agent context

Co-Authored-By: Claude Opus 4.6 <[email protected]>
2026-02-14 11:04:36 -06:00
jason.woltjeandClaude Opus 4.6 0ca3945061 fix(api): resolve Docker startup failures (secrets, Redis, Prisma)
- Pass BETTER_AUTH_SECRET through all 6 docker-compose files to API container
- Fix BullModule to parse VALKEY_URL instead of VALKEY_HOST/VALKEY_PORT,
  matching all other Redis consumers in the codebase
- Migrate Prisma encryption from removed $use() middleware to $extends()
  client extensions (Prisma 6.x compatibility), keeping extends PrismaClient
  pattern with only account and llmProviderInstance getters overridden

Co-Authored-By: Claude Opus 4.6 <[email protected]>
2026-02-14 11:04:04 -06:00
jason.woltjeandClaude Opus 4.6 7b892d5197 fix(api): import AuthModule in FederationModule for DI resolution
AuthGuard used across federation controllers depends on AuthService,
which requires AuthModule to be imported. Matches pattern used by
TasksModule, ProjectsModule, and CredentialsModule.

Co-Authored-By: Claude Opus 4.6 <[email protected]>
2026-02-13 22:36:11 -06:00
jason.woltjeandClaude Opus 4.6 e23490a5f7 fix(api): remove redundant CsrfGuard from FederationController
CsrfGuard is already applied globally via APP_GUARD in AppModule.
The explicit @UseGuards(CsrfGuard) on FederationController caused a
DI error because CsrfService is not provided in FederationModule.

Co-Authored-By: Claude Opus 4.6 <[email protected]>
2026-02-13 22:14:03 -06:00
jason.woltje 1b3ff1b5e1 Merge pull request 'fix(ci): Node.js 20 → 24 LTS + pipeline fixes (#366, #367)' (#368) from fix/ci-366 into develop
Reviewed-on: http://git.mosaicstack.dev/mosaic/stack/pulls/368
2026-02-13 23:18:04 +00:00
jason.woltje 46be7aa36f Merge branch 'develop' into fix/ci-366 2026-02-13 23:17:55 +00:00
jason.woltje 2ab795a95d Merge pull request 'fix(ci): fix pipeline #366 — web @mosaic/ui build, Dockerfile find bug, event handler types' (#366) from fix/ci-366 into develop
Reviewed-on: http://git.mosaicstack.dev/mosaic/stack/pulls/366
2026-02-13 00:27:48 +00:00
jason.woltjeandClaude Opus 4.6 e368083e84 fix(api): import AuthModule in CredentialsModule for DI resolution
CredentialsController uses AuthGuard which depends on AuthService.
NestJS resolves guard dependencies in the module context, so
CredentialsModule needs to import AuthModule directly.

Co-Authored-By: Claude Opus 4.6 <[email protected]>
2026-02-11 21:14:20 -06:00
jason.woltjeandClaude Opus 4.6 4a4d3efbfb fix(ci): move pipeline config into .woodpecker/ directory
Woodpecker v3 ignores .woodpecker.yml when a .woodpecker/ directory
exists, reading only files from the directory. Since develop has
.woodpecker/codex-review.yml, the main build pipeline was invisible
to Woodpecker on develop. Move it into the directory as build.yml.

Co-Authored-By: Claude Opus 4.6 <[email protected]>
2026-02-11 20:58:26 -06:00
jason.woltjeandClaude Opus 4.6 3a922d447f ci: test webhook trigger on develop branch
Co-Authored-By: Claude Opus 4.6 <[email protected]>
2026-02-11 20:57:24 -06:00
jason.woltjeandClaude Opus 4.6 9ff1e69860 chore(api): remove debug statements from Dockerfile
Remove temporary debug RUN layers that were added during initial
build troubleshooting. These add build time and leak directory
structure into build logs unnecessarily.

Co-Authored-By: Claude Opus 4.6 <[email protected]>
2026-02-11 20:54:37 -06:00
jason.woltjeandClaude Opus 4.6 c8bf7f6b70 chore: trigger CI pipeline on develop
Co-Authored-By: Claude Opus 4.6 <[email protected]>
2026-02-11 20:31:24 -06:00
jason.woltjeandClaude Opus 4.6 64396cf9de chore: trigger CI rebuild from current develop HEAD
Co-Authored-By: Claude Opus 4.6 <[email protected]>
2026-02-11 20:30:42 -06:00
jason.woltje 1456a6f149 chore: trigger CI rebuild for develop images 2026-02-11 19:43:44 -06:00
jason.woltje fc2a13ad74 chore: trigger CI pipeline rebuild 2026-02-11 19:42:26 -06:00
jason.woltjeandClaude Opus 4.6 72b1d9f4f2 fix(devops): make OpenBao compose Swarm/Portainer compatible
Convert docker-compose.openbao.yml from standalone Docker Compose
to Swarm-compatible format:
- Remove container_name, depends_on, restart (not supported in Swarm)
- Add deploy.restart_policy sections
- Remove 127.0.0.1 port binding (use overlay network instead)
- Remove env_file (use Portainer environment instead)
- Init sidecar limited to 5 restart attempts with 10s delay

Co-Authored-By: Claude Opus 4.6 <[email protected]>
2026-02-11 19:41:05 -06:00
jason.woltjeandClaude Opus 4.6 b3c0f51dc9 fix(devops): enable OpenBao in Swarm and fix healthchecks
- Enable OpenBao + init sidecar in Swarm compose (was commented out)
- Fix healthcheck to accept uninitialized/sealed vault states
  (add ?uninitcode=200&sealedcode=200 to /v1/sys/health)
- Replace nc-based healthcheck with wget in dev compose
- Add ORCHESTRATOR_URL env var to API service in Swarm compose
- Uncomment OpenBao volumes in Swarm compose

The healthcheck was returning HTTP 501 for uninitialized vault,
causing Swarm to restart OpenBao before init sidecar could run.

Co-Authored-By: Claude Opus 4.6 <[email protected]>
2026-02-11 19:38:34 -06:00
jason.woltjeandClaude Opus 4.6 6a5a4e4de8 feat(web): add credential management UI pages and components
Add credentials settings page, audit log page, CRUD dialog components
(create, view, edit, rotate), credential card, dialog UI component,
and API client for the M7-CredentialSecurity feature.

Refs #346

Co-Authored-By: Claude Opus 4.6 <[email protected]>
2026-02-10 09:42:41 -06:00
jason.woltjeandClaude Opus 4.6 ab64583951 fix: resolve deployment crashes in coordinator and API services
Coordinator: install all dependencies from pyproject.toml instead of
hardcoded subset (missing slowapi, anthropic, opentelemetry-*).

API: FederationAgentService now gracefully disables when orchestrator
URL is not configured instead of throwing and crashing the app.

Co-Authored-By: Claude Opus 4.6 <[email protected]>
2026-02-10 09:41:54 -06:00
jason.woltjeandClaude Opus 4.6 f3694592cc feat(swarm): add coordinator service and reorganize compose files
- Add coordinator service to docker-compose.swarm.portainer.yml and
  docker-compose.swarm.yml with full environment config and healthcheck
- Add ANTHROPIC_API_KEY and coordinator settings to .env.swarm.example
- Move docker-compose.override.yml.example and docker-compose.prod.yml
  into docker/ directory
- Add *.bak to .gitignore

Co-Authored-By: Claude Opus 4.6 <[email protected]>
2026-02-09 22:04:55 -06:00
jason.woltjeandClaude Opus 4.6 c4f6552e12 docs(agents): add AGENTS.md context files for all modules
Adds directory-specific agent context templates for AI-assisted
development across all apps and packages.

Co-Authored-By: Claude Opus 4.6 <[email protected]>
2026-02-09 22:04:43 -06:00
jason.woltjeandClaude Opus 4.6 af2e2b083d feat(ci): add Codex AI review pipeline for Woodpecker
Adds automated code quality and security review pipeline that runs on
pull requests using OpenAI Codex with structured output schemas.

Co-Authored-By: Claude Opus 4.6 <[email protected]>
2026-02-09 22:04:34 -06:00
jason.woltjeandClaude Opus 4.6 281c7ab39b fix(orchestrator): resolve DockerSandboxService DI failure on startup
Add explicit @Inject("DOCKER_CLIENT") token to the Docker constructor
parameter in DockerSandboxService. The @Optional() decorator alone was
not suppressing the NestJS resolution error for the external dockerode
class, causing the orchestrator container to crash on startup.

Co-Authored-By: Claude Opus 4.6 <[email protected]>
2026-02-09 21:22:52 -06:00
jason.woltje d273220838 Merge pull request 'Merge feature/m4-llm-integration into develop' (#362) from feature/m4-llm-integration into develop
Reviewed-on: http://git.mosaicstack.dev/mosaic/stack/pulls/362
2026-02-09 20:17:44 +00:00
jason.woltje 709499c167 fix(api,orchestrator): fix remaining dependency injection issues
API:
- Add AuthModule import to JobEventsModule
- Add AuthModule import to JobStepsModule
- Fixes: AuthGuard dependency resolution in job modules

Orchestrator:
- Add @Optional() decorator to docker parameter in DockerSandboxService
- Fixes: NestJS trying to inject Docker class as dependency

All modules using AuthGuard must import AuthModule.
Docker parameter is optional for testing, needs @Optional() decorator.
2026-02-08 22:24:37 -06:00
jason.woltje ecfd02541f fix(test): add VaultService dependencies to job-events performance test
- Add ConfigService mock for encryption configuration
- Add VaultService and CryptoService to test module
- Fixes: PrismaService dependency injection error in test

PrismaService requires VaultService for credential encryption.
Performance tests now properly provide all required dependencies.

Refs #341 (pipeline test failure)
2026-02-08 22:04:24 -06:00
jason.woltje 4545c6dc7a fix(api,orchestrator): fix dependency injection and Docker build issues
API:
- Add AuthModule import to RunnerJobsModule
- Fixes: Nest can't resolve dependencies of AuthGuard

Orchestrator:
- Remove --prod flag from dependency installation
- Copy full node_modules tree to production stage
- Align Dockerfile with API pattern for monorepo builds
- Fixes: Cannot find module '@nestjs/core'

Both services now match the working API Dockerfile pattern.
2026-02-08 21:59:19 -06:00
jason.woltjeandClaude Opus 4.6 3485ab7883 fix(swarm): remove postgres init-scripts bind mount for Portainer
- Remove ./docker/postgres/init-scripts bind mount from postgres service
- Fixes: 'bind source path does not exist' error in Portainer
- Init scripts are already baked into postgres image at build time

Portainer can't access repository files when deploying stacks,
so bind mounts to local paths don't work. The postgres image
already includes init scripts via Dockerfile COPY.

Co-Authored-By: Claude Opus 4.6 <[email protected]>
2026-02-08 20:29:25 -06:00
jason.woltjeandClaude Opus 4.6 66269fa816 feat(portainer): add Portainer-optimized deployment files
- Create docker-compose.portainer.yml
  - No env_file directive (Portainer doesn't support it)
  - Port exposed on 0.0.0.0 (Portainer limitation)
  - Simple depends_on syntax
  - All environment variables explicit

- Create docs/PORTAINER-DEPLOYMENT.md
  - Complete Portainer deployment guide
  - Step-by-step instructions
  - Environment variables reference
  - Troubleshooting section
  - Best practices for security and backups

- Update README.md
  - Add Portainer deployment section
  - Reference Portainer deployment guide

Fixes:
- 'open /data/compose/94/.env: no such file or directory'
- 'ignoring IP-address (127.0.0.1:8200:8200/tcp)' warning

Portainer requires different compose syntax than standard docker-compose.
This provides a deployment path optimized for Portainer's stack parser.

Co-Authored-By: Claude Opus 4.6 <[email protected]>
2026-02-08 17:41:11 -06:00
jason.woltjeandClaude Opus 4.6 83dee62f0e fix(openbao): use simple depends_on syntax for Portainer compatibility
- Change depends_on from condition-based to simple list syntax
- Fixes: 'Services.openbao-init.depends_on must be a list' error
- Compatible with Portainer's compose parser

Co-Authored-By: Claude Opus 4.6 <[email protected]>
2026-02-08 17:38:40 -06:00
jason.woltjeandClaude Opus 4.6 7c01352ab5 fix(openbao): use production mode instead of dev mode
- Add explicit command: server -config=/openbao/config/config.hcl
- Remove OPENBAO_DEV_ROOT_TOKEN_ID (not needed in production)
- Fixes 'address already in use' error caused by dev mode conflict

The base OpenBao image defaults to 'server -dev' which conflicts with
our production config.hcl. This change forces production mode.

Co-Authored-By: Claude Opus 4.6 <[email protected]>
2026-02-08 17:34:36 -06:00
jason.woltjeandClaude Opus 4.6 c195b8c8fd feat(openbao): add standalone deployment for swarm compatibility
- Create docker-compose.openbao.yml for standalone OpenBao deployment
  - Includes openbao and openbao-init services
  - Auto-initialization on first run
  - Connects to swarm's mosaic_internal network
  - Binds to localhost:8200 for security

- Update docker-compose.swarm.yml
  - Comment out OpenBao service (cannot run in swarm)
  - Add clear note about standalone requirement
  - Update volumes section
  - Update header with current config

- Create docs/OPENBAO-DEPLOYMENT.md
  - Comprehensive deployment guide
  - 4 deployment options: standalone, bundled, external, fallback
  - Clear explanation why OpenBao can't run in swarm
  - Deployment workflows for each scenario
  - Troubleshooting section

- Update docs/SWARM-DEPLOYMENT.md
  - Add Step 1: Deploy OpenBao standalone FIRST
  - Remove manual initialization (now automatic)
  - Update expected services list
  - Reference OpenBao deployment guide

- Update README.md
  - Clarify OpenBao standalone requirement for swarm
  - Update deployment steps
  - Highlight critical requirement at top of notes

Key changes:
- OpenBao MUST be deployed standalone when using swarm
- Automatic initialization via openbao-init sidecar
- Clear documentation for all deployment options
- Swarm stack no longer includes OpenBao

Co-Authored-By: Claude Opus 4.6 <[email protected]>
2026-02-08 17:30:30 -06:00
jason.woltjeandClaude Opus 4.6 dac735af56 fix(swarm): move docker-compose.swarm.yml back to root directory
- Move docker/docker-compose.swarm.yml to root
- Update documentation references
- Simplifies deployment: swarm file in root, standalone file in root
- Deploy script already expects file in root

Rationale: Keep it simple - two compose files for two deployment methods:
  - docker-compose.yml → standalone (docker compose up -d)
  - docker-compose.swarm.yml → swarm (docker stack deploy)

Co-Authored-By: Claude Opus 4.6 <[email protected]>
2026-02-08 17:22:20 -06:00
jason.woltjeandClaude Opus 4.6 f8477d5052 docs(swarm): comprehensive Docker Swarm deployment documentation
- Update docker-compose.swarm.yml with external Authentik configuration
  - Comment out Authentik services (using external OIDC provider)
  - Comment out Authentik volumes
  - Add header with deployment instructions and current configuration

- Create comprehensive SWARM-DEPLOYMENT.md guide
  - Prerequisites and swarm initialization
  - Manual OpenBao initialization (critical - no auto-init in swarm)
  - External service configuration examples
  - Scaling, updates, rollbacks
  - Troubleshooting and maintenance procedures
  - Backup and restore instructions

- Update .env.swarm.example
  - Add note about external vs internal Authentik
  - Update default OIDC_ISSUER to use https
  - Clarify which variables are needed for internal Authentik

- Update README.md Docker Swarm section
  - Fix deploy script path (./scripts/deploy-swarm.sh)
  - Add note about manual OpenBao initialization
  - Add warning about no profile support in swarm
  - Update documentation references to docs/ directory

- Update documentation cross-references
  - Add deprecation notice to old DOCKER-SWARM.md
  - Add deployment guide reference to SWARM-QUICKREF.md
  - Update DOCKER-COMPOSE-GUIDE.md See Also section

Key changes for swarm deployment:
- Swarm does NOT support docker-compose profiles
- External services must be manually commented out
- OpenBao requires manual initialization (no sidecar)
- All documentation updated with correct paths

Co-Authored-By: Claude Opus 4.6 <[email protected]>
2026-02-08 17:12:49 -06:00
jason.woltjeandClaude Opus 4.6 6521cba735 feat: add flexible docker-compose architecture with profiles
- Add OpenBao services to docker-compose.yml with profiles (openbao, full)
- Add docker-compose.build.yml for local builds vs registry pulls
- Make PostgreSQL and Valkey optional via profiles (database, cache)
- Create example compose files for common deployment scenarios:
  - docker/docker-compose.example.turnkey.yml (all bundled)
  - docker/docker-compose.example.external.yml (all external)
  - docker/docker.example.hybrid.yml (mixed deployment)
- Update documentation:
  - Enhance .env.example with profiles and external service examples
  - Update README.md with deployment mode quick starts
  - Add deployment scenarios to docs/OPENBAO.md
  - Create docker/DOCKER-COMPOSE-GUIDE.md with comprehensive guide
- Clean up repository structure:
  - Move shell scripts to scripts/ directory
  - Move documentation to docs/ directory
  - Move docker compose examples to docker/ directory
- Configure for external Authentik with internal services:
  - Comment out Authentik services (using external OIDC)
  - Comment out unused volumes for disabled services
  - Keep postgres, valkey, openbao as internal services

This provides a flexible deployment architecture supporting turnkey,
production (all external), and hybrid configurations via Docker Compose
profiles.

Co-Authored-By: Claude Opus 4.6 <[email protected]>
2026-02-08 16:55:33 -06:00
jason.woltjeandClaude Opus 4.6 71b32398ad fix(ci): Add set -e to link-packages for proper error propagation
Without set -e, if an individual link_package call fails, the script
continues silently. Only the last call's exit code determined the step
result — masking earlier failures.

Co-Authored-By: Claude Opus 4.6 <[email protected]>
2026-02-08 15:29:23 -06:00
jason.woltjeandClaude Opus 4.6 c5b028932c fix(ci): Add retry logic for package linking with delay
Addresses timing issue where packages aren't immediately queryable via
API after being pushed to the registry.

Changes:
- Initial 10-second delay for package indexing
- Retry logic: 3 attempts with 5-second delays
- Only retries on 404 (not found) errors
- Returns success on 201/204 (linked) or 400 (already linked)
- Better logging shows attempt progress

This fixes the race condition where link-packages ran before packages
were indexed in Gitea's registry API.

Co-Authored-By: Claude Opus 4.6 <[email protected]>
2026-02-08 15:04:55 -06:00
jason.woltje 5b5a5e458a test(ci): Minimal pipeline to test package linking variable expansion 2026-02-08 15:00:32 -06:00
jason.woltjeandClaude Opus 4.6 f1e6fc29f6 fix(ci): Escape dollar signs for shell variables in Woodpecker
Woodpecker interprets $ as variable substitution in YAML, so we need to
use $$ to escape it and pass a literal $ to the shell script.

Changed from a for loop to explicit function calls with escaped variables:
- Use $$ instead of $ for all shell variables
- Function-based approach for cleaner variable passing
- Each package explicitly called: link_package "stack-api" etc.

This fixes the variable expansion issue where ${package} was empty,
resulting in URLs like "container//-/link/stack" (double slash).

Co-Authored-By: Claude Opus 4.6 <[email protected]>
2026-02-08 14:58:15 -06:00
jason.woltjeandClaude Opus 4.6 aad6cb75d0 fix(ci): Handle 201 status code for package linking
The Gitea package link API returns 201 (Created) on successful linking,
not 204 (No Content) as we were checking for. Updated the link-packages
step to accept both 201 and 204 as success.

Also added visual indicators (/) to make link status clearer in logs.

Diagnostic output showed all 5 packages successfully linked with 201:
- stack-api: 201 (linked)
- stack-web: 201 (linked)
- stack-postgres: 201 (linked)
- stack-openbao: 201 (linked)
- stack-orchestrator: 201 (linked)

Subsequent runs return 400 "invalid argument" which means already linked.

Co-Authored-By: Claude Opus 4.6 <[email protected]>
2026-02-08 14:46:48 -06:00
jason.woltjeandClaude Opus 4.6 a61f9262e6 fix(ci): Add missing OpenBao Dockerfile
The docker-build-openbao pipeline step was failing because the Dockerfile
was missing from docker/openbao/.

Created a minimal Dockerfile that:
- Uses official quay.io/openbao/openbao:2 as base
- Copies config.hcl and init.sh into the image
- Exposes port 8200
- Preserves the default entrypoint from base image

This allows Kaniko to build the stack-openbao image for Swarm deployment.

Fixes pipeline #325 docker-build-openbao failure.

Co-Authored-By: Claude Opus 4.6 <[email protected]>
2026-02-08 02:20:02 -06:00
jason.woltjeandClaude Opus 4.6 32aff3787d fix(test): Fix FilterBar and TaskList test failures
FilterBar Test Fix:
- Skip onFilterChange callback on first render to prevent spurious calls
- Use isFirstRender ref to track initial mount
- Prevents "expected spy to not be called" failure in debounce test

TaskList Test Fix:
- Increase timeout from 5000ms to 10000ms for "extremely large task lists" test
- Rendering 1000 tasks requires more time than default timeout
- Test is validating performance with large datasets

These fixes resolve pipeline #324 test failures.

Co-Authored-By: Claude Opus 4.6 <[email protected]>
2026-02-08 02:09:40 -06:00
jason.woltje 8b78ffe4a0 refactor(ci): Rename images to stack-* prefix for clarity
Renamed all Docker images from generic names to stack-* prefix:
- api → stack-api
- web → stack-web
- postgres → stack-postgres
- openbao → stack-openbao
- orchestrator → stack-orchestrator

This prevents confusion with other repositories in the mosaic/
organization on git.mosaicstack.dev.

Registry images:
  git.mosaicstack.dev/mosaic/stack-api
  git.mosaicstack.dev/mosaic/stack-web
  git.mosaicstack.dev/mosaic/stack-postgres
  git.mosaicstack.dev/mosaic/stack-openbao
  git.mosaicstack.dev/mosaic/stack-orchestrator

Local images:
  stack-api:latest
  stack-web:latest
  stack-postgres:latest
  stack-openbao:latest
  stack-orchestrator:latest

Updated files:
- .woodpecker.yml (all build steps + package linking)
- docker-compose.swarm.yml (all image references)
- build-images.sh (local image names)
- deploy-swarm.sh (image validation)
2026-02-08 02:03:31 -06:00
jason.woltjeandClaude Opus 4.6 f0bfbe4367 fix: Use POST for Gitea package link API and handle already-linked
The link endpoint uses POST (not PUT) and returns 400 when already
linked. Handle both 204 (linked) and 400 (already linked) as success.

Co-Authored-By: Claude Opus 4.6 <[email protected]>
2026-02-08 02:02:15 -06:00
jason.woltje 657c33927b feat(ci): Add package linking to repository
Link all Docker container packages to the mosaic/stack repository
using Gitea's package API. This makes packages visible on the
repository page and shows which repo they came from.

API endpoint: /packages/{owner}/container/{name}/-/link/{repo_name}

Links created for:
- mosaic/api
- mosaic/web
- mosaic/postgres
- mosaic/openbao
- mosaic/orchestrator

Each package will now show up in the repository's packages tab.
2026-02-08 01:59:19 -06:00
jason.woltjeandClaude Opus 4.6 2ca36b1518 fix(test): Use real timers for FilterBar debounce test
The debounce test was failing in CI because fake timers caused a
deadlock with React's internal rendering timers. Switched to using
real timers with a shorter debounce period (100ms) to make the test
both reliable and fast.

The test now:
- Uses real timers instead of fake timers
- Tests debounce behavior with rapid typing
- Verifies the callback is only called once after debounce completes
- Runs quickly (~100ms) without flakiness

Fixes the CI failure: "expected spy to not be called at all, but
actually been called 1 times"

Co-Authored-By: Claude Opus 4.6 <[email protected]>
2026-02-08 01:55:52 -06:00
jason.woltjeandClaude Opus 4.6 ee6929fad5 fix(test): Fix FilterBar debounce test timing
The "should debounce search input" test was failing because it was
being called immediately instead of after the debounce delay. Fixed by:

1. Using real timers with waitFor instead of fake timers
2. Adding mockOnFilterChange.mockClear() after render to ignore any
   calls from the initial render
3. Properly waiting for the debounced callback with waitFor

This allows the test to correctly verify that:
- The callback is not called immediately after typing
- The callback is called after the 300ms debounce delay
- The callback receives the correct search value

All 19 FilterBar tests now pass.

Co-Authored-By: Claude Opus 4.6 <[email protected]>
2026-02-08 01:46:56 -06:00
jason.woltje 0e3baae415 feat(ci): Add OpenBao and Orchestrator image builds to Woodpecker CI
Add missing Docker image builds for swarm deployment.

Changes:
- Added docker-build-openbao step to .woodpecker.yml
- Added docker-build-orchestrator step to .woodpecker.yml
- Updated docker-compose.swarm.yml to use registry images
  (git.mosaicstack.dev/mosaic/*)
- Added IMAGE_TAG variable support for versioned deployments
- Updated deploy-swarm.sh to support both registry and local images

Image tagging strategy:
- All commits: SHA tag (e.g., 658ec077)
- main branch: latest + SHA
- develop branch: dev + SHA
- git tags: version tag + SHA

Registry images:
- git.mosaicstack.dev/mosaic/postgres
- git.mosaicstack.dev/mosaic/openbao
- git.mosaicstack.dev/mosaic/api
- git.mosaicstack.dev/mosaic/orchestrator
- git.mosaicstack.dev/mosaic/web

Deployment modes:
- IMAGE_TAG=latest (default, use registry latest)
- IMAGE_TAG=dev (use registry dev tag)
- IMAGE_TAG=local (use local builds via build-images.sh)
2026-02-08 01:33:36 -06:00
jason.woltje 7f3499b1f2 fix(swarm): Remove build directives and unsupported options for swarm
Docker Swarm doesn't support build directives or security_opt.
Images must be pre-built before deployment.

Changes:
- Created build-images.sh script to build all images
- Updated deploy-swarm.sh to check for images and offer to build
- Removed build: sections from docker-compose.swarm.yml
- Removed security_opt: (not supported in swarm)
- Services now reference pre-built images only

Deployment workflow:
1. ./build-images.sh (build all images)
2. ./deploy-swarm.sh mosaic (deploy to swarm)
2026-02-08 01:31:29 -06:00
jason.woltje 2a9a1f1367 fix(swarm): Convert boolean env vars to strings in orchestrator service
Docker Compose/Swarm requires environment variables to be strings, not booleans.

Changes:
- KILLSWITCH_ENABLED: true -> "true"
- SANDBOX_ENABLED: true -> "true"

Fixes deployment error: 'must be a string, number or null'
2026-02-08 01:30:07 -06:00
jason.woltje ed92bb5402 feat(#swarm): Add Docker Swarm deployment with AI provider configuration
- Add setup-wizard.sh for interactive configuration
- Add docker-compose.swarm.yml optimized for swarm deployment
- Make CLAUDE_API_KEY optional based on AI_PROVIDER setting
- Support multiple AI providers: Ollama, Claude API, OpenAI
- Add BETTER_AUTH_SECRET to .env.example
- Update deploy-swarm.sh to validate AI provider config
- Add comprehensive documentation (DOCKER-SWARM.md, SWARM-QUICKREF.md)

Changes:
- AI_PROVIDER env var controls which AI backend to use
- Ollama is default (no API key required)
- Claude API and OpenAI require respective API keys
- Deployment script validates based on selected provider
- Removed Authentik services from swarm compose (using external)
- Configured for upstream Traefik integration
2026-02-08 01:18:04 -06:00
jason.woltjeandClaude Opus 4.6 dc551f138a fix(test): Use correct CI detection for Woodpecker
Woodpecker sets CI=woodpecker and CI_PIPELINE_EVENT, not CI=true.
Updated the CI detection to check for both.

Co-Authored-By: Claude Opus 4.6 <[email protected]>
2026-02-07 21:47:53 -06:00
jason.woltjeandClaude Opus 4.6 75766a37b4 fix(test): Skip loading .env.test in CI environments
The .env.test file was being loaded in CI and overriding the CI-provided
DATABASE_URL, causing tests to try connecting to localhost:5432 instead of
the postgres:5432 service.

Fix: Only load .env.test when NOT in CI (check for CI or WOODPECKER env vars).

Co-Authored-By: Claude Opus 4.6 <[email protected]>
2026-02-07 21:44:02 -06:00
jason.woltjeandClaude Opus 4.6 0b0666558e fix(test): Fix DATABASE_URL environment setup for integration tests
Fixes integration test failures caused by missing DATABASE_URL environment variable.

Changes:
- Add dotenv as dev dependency to load .env.test in vitest setup
- Add .env.test to .gitignore to prevent committing test credentials
- Create .env.test.example with warning comments for documentation
- Add conditional test skipping when DATABASE_URL is not available
- Add DATABASE_URL format validation in vitest setup
- Add error handling to test cleanup to prevent silent failures
- Remove filesystem path disclosure from error messages

The fix allows integration tests to:
- Load DATABASE_URL from .env.test locally for developers with database setup
- Skip gracefully if DATABASE_URL is not available (no database running)
- Connect to postgres service in CI where DATABASE_URL is explicitly provided

Tests affected: auth-rls.integration.spec.ts and other integration tests
requiring real database connections.

Co-Authored-By: Claude Opus 4.6 <[email protected]>
2026-02-07 17:46:59 -06:00
jason.woltje 4552c2c460 fix(test): Add ENCRYPTION_KEY to bridge.module.spec.ts and fix API lint errors 2026-02-07 17:33:32 -06:00
jason.woltje b9e1e3756e fix(ci): Add ENCRYPTION_KEY to test environment 2026-02-07 17:28:15 -06:00
jason.woltje 9f0956d4a4 chore: M9-CredentialSecurity milestone COMPLETE - All 12 issues closed 2026-02-07 17:24:14 -06:00
jason.woltjeandClaude Opus 4.6 73074932f6 feat(#360): Add federation credential isolation
Implement explicit deny-lists in QueryService and CommandService to prevent
user credentials from leaking across federation boundaries.

## Changes

### Core Implementation
- QueryService: Block all credential-related queries with keyword detection
- CommandService: Block all credential operations (create/update/delete/read)
- Case-insensitive keyword matching for both queries and commands

### Security Features
- Deny-list includes: credential, api_key, secret, token, password, oauth
- Errors returned for blocked operations
- No impact on existing allowed operations (tasks, events, projects, agent commands)

### Testing
- Added 2 unit tests to query.service.spec.ts
- Added 3 unit tests to command.service.spec.ts
- Added 8 integration tests in credential-isolation.integration.spec.ts
- All 377 federation tests passing

### Documentation
- Created comprehensive security doc at docs/security/federation-credential-isolation.md
- Documents 4 security guarantees (G1-G4)
- Includes testing strategy and incident response procedures

## Security Guarantees

1. G1: Credential Confidentiality - Credentials never leave instance in plaintext
2. G2: Cross-Instance Isolation - Compromised key on one instance doesn't affect others
3. G3: Query/Command Isolation - Federated instances cannot query/modify credentials
4. G4: Accidental Exposure Prevention - Credentials cannot leak via messages

## Defense-in-Depth

This implementation adds application-layer protection on top of existing:
- Transit key separation (mosaic-credentials vs mosaic-federation)
- Per-instance OpenBao servers
- Workspace-scoped credential access

Fixes #360

Co-Authored-By: Claude Opus 4.6 <[email protected]>
2026-02-07 16:55:49 -06:00
jason.woltje 33dc746714 chore: Update tasks.md - Issues #356 and #359 complete 2026-02-07 16:51:05 -06:00
jason.woltje 46d0a06ef5 feat(#356): Build credential CRUD API endpoints
Implement comprehensive CRUD API for managing user credentials with encryption,
RLS, and audit logging following TDD methodology.

Features:
- POST /api/credentials - Create encrypted credential
- GET /api/credentials - List credentials (masked values only)
- GET /api/credentials/:id - Get single credential (masked)
- GET /api/credentials/:id/value - Decrypt plaintext (rate limited 10/min)
- PATCH /api/credentials/:id - Update metadata
- POST /api/credentials/:id/rotate - Rotate credential value
- DELETE /api/credentials/:id - Soft delete

Security:
- All values encrypted via VaultService (TransitKey.CREDENTIALS)
- List/Get endpoints NEVER return plaintext (only maskedValue)
- getValue endpoint rate limited to 10 requests/minute per user
- All operations audit-logged with CREDENTIAL_* ActivityAction
- RLS enforces per-user isolation via getRlsClient() pattern
- Input validation via class-validator DTOs

Testing:
- 26/26 unit tests passing
- 95.71% code coverage (exceeds 85% requirement)
  - Service: 95.16%
  - Controller: 100%
- TypeScript checks pass

Files created:
- apps/api/src/credentials/credentials.service.ts
- apps/api/src/credentials/credentials.service.spec.ts
- apps/api/src/credentials/credentials.controller.ts
- apps/api/src/credentials/credentials.controller.spec.ts
- apps/api/src/credentials/credentials.module.ts
- apps/api/src/credentials/dto/*.dto.ts (5 DTOs)

Files modified:
- apps/api/src/app.module.ts - imported CredentialsModule

Note: Admin credentials endpoints deferred to future issue. Current
implementation covers all user credential endpoints.

Refs #346
Co-Authored-By: Claude Opus 4.6 <[email protected]>
2026-02-07 16:50:02 -06:00
jason.woltjeandClaude Opus 4.6 aa2ee5aea3 feat(#359): Encrypt LLM provider API keys in database
Implemented transparent encryption/decryption of LLM provider API keys
stored in llm_provider_instances.config JSON field using OpenBao Transit
encryption.

Implementation:
- Created llm-encryption.middleware.ts with encryption/decryption logic
- Auto-detects format (vault:v1: vs plaintext) for backward compatibility
- Idempotent encryption prevents double-encryption
- Registered middleware in PrismaService
- Created data migration script for active encryption
- Added migrate:encrypt-llm-keys command to package.json

Tests:
- 14 comprehensive unit tests
- 90.76% code coverage (exceeds 85% requirement)
- Tests create, read, update, upsert operations
- Tests error handling and backward compatibility

Migration:
- Lazy migration: New keys encrypted, old keys work until re-saved
- Active migration: pnpm --filter @mosaic/api migrate:encrypt-llm-keys
- No schema changes required
- Zero downtime

Security:
- Uses TransitKey.LLM_CONFIG from OpenBao Transit
- Keys never touch disk in plaintext (in-memory only)
- Transparent to LlmManagerService and providers
- Follows proven pattern from account-encryption.middleware.ts

Files:
- apps/api/src/prisma/llm-encryption.middleware.ts (new)
- apps/api/src/prisma/llm-encryption.middleware.spec.ts (new)
- apps/api/scripts/encrypt-llm-keys.ts (new)
- apps/api/prisma/migrations/20260207_encrypt_llm_api_keys/ (new)
- apps/api/src/prisma/prisma.service.ts (modified)
- apps/api/package.json (modified)

Note: The migration script (encrypt-llm-keys.ts) is not included in
tsconfig.json to avoid rootDir conflicts. It's executed via tsx which
handles TypeScript directly.

Refs #359

Co-Authored-By: Claude Opus 4.6 <[email protected]>
2026-02-07 16:49:37 -06:00
jason.woltjeandClaude Opus 4.6 864c23dc94 feat(#355): Create UserCredential model with RLS and encryption support
Implements secure user credential storage with comprehensive RLS policies
and encryption-ready architecture for Phase 3 of M9-CredentialSecurity.

**Features:**
- UserCredential Prisma model with 19 fields
- CredentialType enum (6 values: API_KEY, OAUTH_TOKEN, etc.)
- CredentialScope enum (USER, WORKSPACE, SYSTEM)
- FORCE ROW LEVEL SECURITY with 3 policies
- Encrypted value storage (OpenBao Transit ready)
- Cascade delete on user/workspace deletion
- Activity logging integration (CREDENTIAL_* actions)
- 28 comprehensive test cases

**Security:**
- RLS owner bypass, user access, workspace admin policies
- SQL injection hardening for is_workspace_admin()
- Encryption version tracking ready
- Full down migration for reversibility

**Testing:**
- 100% enum coverage (all CredentialType + CredentialScope values)
- Unique constraint enforcement
- Foreign key cascade deletes
- Timestamp behavior validation
- JSONB metadata storage

**Files:**
- Migration: 20260207_add_user_credentials (184 lines + 76 line down.sql)
- Security: 20260207163740_fix_sql_injection_is_workspace_admin
- Tests: user-credential.model.spec.ts (28 tests, 544 lines)
- Docs: README.md (228 lines), scratchpad

Fixes #355

Co-Authored-By: Claude Opus 4.6 <[email protected]>
2026-02-07 16:39:15 -06:00
jason.woltje 1f86c36cc1 chore: Update tasks.md - Phase 2 complete (3/3) 2026-02-07 16:17:51 -06:00
jason.woltjeandClaude Sonnet 4.5 40f7e7e4c0 docs(#354): Add comprehensive OpenBao integration guide
Complete documentation for OpenBao Transit encryption covering setup,
architecture, production hardening, and operations.

Sections:
- Overview: Why OpenBao, Transit encryption explained
- Architecture: Data flow diagrams, fallback behavior
- Default Setup: Turnkey auto-init/unseal, file locations
- Environment Variables: Configuration options
- Transit Keys: Named keys, rotation procedures
- Production Hardening: 10-point security checklist
- Operations: Health checks, manual procedures, monitoring
- Troubleshooting: Common issues and solutions
- Disaster Recovery: Backup/restore procedures

Key Topics:
- Shamir key splitting upgrade (1-of-1 → 3-of-5)
- TLS configuration for production
- Audit logging enablement
- HA storage backends (Raft/Consul)
- External auto-unseal with KMS
- Rate limiting via reverse proxy
- Network isolation best practices
- Key rotation procedures
- Backup automation

Closes #354

Co-Authored-By: Claude Sonnet 4.5 <[email protected]>
2026-02-07 16:16:51 -06:00
jason.woltjeandClaude Opus 4.6 dd171b287f feat(#353): Create VaultService NestJS module for OpenBao Transit
Implements secure credential encryption using OpenBao Transit API with
automatic fallback to AES-256-GCM when OpenBao is unavailable.

Features:
- AppRole authentication with automatic token renewal at 50% TTL
- Transit encrypt/decrypt with 4 named keys
- Automatic fallback to CryptoService when OpenBao unavailable
- Auto-detection of ciphertext format (vault:v1: vs AES)
- Request timeout protection (5s default)
- Health indicator for monitoring
- Backward compatible with existing AES-encrypted data

Security:
- ERROR-level logging for fallback
- Proper error propagation (no silent failures)
- Request timeouts prevent hung operations
- Secure credential file reading

Migrations:
- Account encryption middleware uses VaultService
- Uses TransitKey.ACCOUNT_TOKENS for OAuth tokens
- Backward compatible with existing encrypted data

Tests: 56 tests passing (36 VaultService + 20 middleware)

Closes #353

Co-Authored-By: Claude Opus 4.6 <[email protected]>
2026-02-07 16:13:05 -06:00
jason.woltjeandClaude Opus 4.6 d4d1e59885 feat(#357): Add OpenBao to Docker Compose with turnkey setup
Implements secure credential storage using OpenBao Transit encryption.

Features:
- Auto-initialization on first run (1-of-1 Shamir key for dev)
- Auto-unseal on container restart with verification and retry logic
- Transit secrets engine with 4 named encryption keys
- AppRole authentication with Transit-only policy
- Localhost-only API binding for security
- Comprehensive integration test suite (22 tests, all passing)

Security:
- API bound to 127.0.0.1 (localhost only, no external access)
- Unseal verification with 3-attempt retry logic
- Sanitized error messages in tests (no secret leakage)
- Volume-based secret reading (doesn't require running container)

Files:
- docker/openbao/config.hcl: Server configuration
- docker/openbao/init.sh: Auto-init/unseal script
- docker/docker-compose.yml: OpenBao and init services
- tests/integration/openbao.test.ts: Full test coverage
- .env.example: OpenBao configuration variables

Closes #357

Co-Authored-By: Claude Opus 4.6 <[email protected]>
2026-02-07 15:40:24 -06:00
jason.woltje 9446475ea2 chore: Update tasks.md - Phase 1 complete (3/3) 2026-02-07 13:17:12 -06:00
jason.woltjeandClaude Opus 4.6 737eb40d18 feat(#352): Encrypt existing plaintext Account tokens
Implements transparent encryption/decryption of OAuth tokens via Prisma middleware with progressive migration strategy.

Core Implementation:
- Prisma middleware transparently encrypts tokens on write, decrypts on read
- Auto-detects ciphertext format: aes:iv:authTag:encrypted, vault:v1:..., or plaintext
- Uses existing CryptoService (AES-256-GCM) for encryption
- Progressive encryption: tokens encrypted as they're accessed/refreshed
- Zero-downtime migration (schema change only, no bulk data migration)

Security Features:
- Startup key validation prevents silent data loss if ENCRYPTION_KEY changes
- Secure error logging (no stack traces that could leak sensitive data)
- Graceful handling of corrupted encrypted data
- Idempotent encryption prevents double-encryption
- Future-proofed for OpenBao Transit encryption (Phase 2)

Token Fields Encrypted:
- accessToken (OAuth access tokens)
- refreshToken (OAuth refresh tokens)
- idToken (OpenID Connect ID tokens)

Backward Compatibility:
- Existing plaintext tokens readable (encryptionVersion = NULL)
- Progressive encryption on next write
- BetterAuth integration transparent (middleware layer)

Test Coverage:
- 20 comprehensive unit tests (89.06% coverage)
- Encryption/decryption scenarios
- Null/undefined handling
- Corrupted data handling
- Legacy plaintext compatibility
- Future vault format support
- All CRUD operations (create, update, updateMany, upsert)

Files Created:
- apps/api/src/prisma/account-encryption.middleware.ts
- apps/api/src/prisma/account-encryption.middleware.spec.ts
- apps/api/prisma/migrations/20260207_encrypt_account_tokens/migration.sql

Files Modified:
- apps/api/src/prisma/prisma.service.ts (register middleware)
- apps/api/src/prisma/prisma.module.ts (add CryptoService)
- apps/api/src/federation/crypto.service.ts (add key validation)
- apps/api/prisma/schema.prisma (add encryptionVersion)
- .env.example (document ENCRYPTION_KEY)

Fixes #352

Co-Authored-By: Claude Opus 4.6 <[email protected]>
2026-02-07 13:16:43 -06:00
jason.woltje 89464583a4 chore: Update tasks.md - Issue #350 complete 2026-02-07 12:49:57 -06:00