Skip to content

Phase 2: normalize environment mode and message-prefix dev parsing#24

Merged
cgrpa merged 1 commit into
mainfrom
secrets
May 13, 2026
Merged

Phase 2: normalize environment mode and message-prefix dev parsing#24
cgrpa merged 1 commit into
mainfrom
secrets

Conversation

@cgrpa

@cgrpa cgrpa commented May 13, 2026

Copy link
Copy Markdown
Owner

Summary

  • implement issue PRD: Remote Secret Loading via Azure Key Vault (Staging First) #22 phase-2 runtime mode cleanup with a shared HostEnvironmentMode contract
  • switch development command handling from bot-prefix mutation to message-prefix parsing (test- applied at parse time)
  • keep canonical bot identities (gemini, grok) and route dev messages via BotRegistry(messagePrefix)
  • harden Terraform secret bootstrap workflow and docs (upsert-required-secrets.sh, Terraform outputs/readme)
  • refresh root/app docs for DSharpPlusBot naming + Development environment behavior

Key Changes

  • add RuntimeMode + HostEnvironmentMode behavior tests
  • refactor DiscordWorker to inject message prefix into BotRegistry
  • refactor BotRegistry.TryGetBot to enforce optional message prefix and then parse canonical bot prefix
  • remove env-prefix constructors from bot config classes and update related tests

Validation

  • dotnet build src/dotnet/TheSexy6BotWorker.Tests/TheSexy6BotWorker.Tests.csproj --no-restore
  • DOTNET_ROLL_FORWARD=Major dotnet test src/dotnet/TheSexy6BotWorker.Tests/TheSexy6BotWorker.Tests.csproj --filter "FullyQualifiedName~HostEnvironmentModeTests|FullyQualifiedName~BotRegistryTests|FullyQualifiedName~BotConfigurationIntegrationTests|FullyQualifiedName~GeminiBotConfigurationTests|FullyQualifiedName~GrokBotConfigurationTests|FullyQualifiedName~BotConfigurationExtensionsTests|FullyQualifiedName~MarkdownIntegrationTests"
  • full DOTNET_ROLL_FORWARD=Major dotnet test src/dotnet/TheSexy6BotWorker.Tests/TheSexy6BotWorker.Tests.csproj still has existing Perplexity integration failures (401 Unauthorized) pending local key refresh

@cgrpa cgrpa merged commit 44d3a84 into main May 13, 2026
6 checks passed
@cgrpa cgrpa deleted the secrets branch May 13, 2026 16:58
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.

1 participant