Skip to content

[P2][cleanup] Remove SHUGUR_* legacy aliases after production migration gate #111

Description

@psam21

Migrated from docs/pending-work-plan.md §7 + docs/Shugur-Migration-Plan.md Phase 6.

Goal

Remove SHUGUR_* fallbacks once the production migration gate is satisfied. This is optional and deferred — it is not part of the immediate branding work.

Gate to remove aliases

The alias-removal gate requires ALL of the following:

  1. A documented production inventory confirming every known production system, local deployment script, Compose environment, CI job, and operator documentation uses NOSTR_* variables.
  2. At least one successful production deployment using only canonical variables.
  3. A rollback test that restores the previous SHUGUR_*-compatible behavior.
  4. An announced deprecation period has elapsed.

Until those conditions are satisfied, the fallback logic is a deliberate compatibility feature rather than unfinished cleanup.

Required changes (when gate is satisfied)

  1. Remove SHUGUR_* environment variable fallback in relay/internal/config/config.go.
  2. Remove the deprecation warning.
  3. Update fresh-install templates to use NOSTR_* only.
  4. Update tests to remove legacy-alias coverage.
  5. Publish a release note documenting the breaking change.
  6. Run the full validation matrix from docs/Shugur-Migration-Plan.md §8.

Acceptance criteria

  • All four gate conditions recorded in the issue with evidence.
  • SHUGUR_* references removed from configuration code.
  • Validation matrix passes from a clean checkout.
  • Release note published.

Dependencies

Source

Migrated from docs/pending-work-plan.md §7 and docs/Shugur-Migration-Plan.md Phase 6.

Metadata

Metadata

Assignees

No one assigned

    Projects

    No projects

      Milestone

      No milestone

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions