Skip to content

fix(sites): remove stale domain links on sync - #110

Merged
ralyodio merged 1 commit into
profullstack:mainfrom
rissrice2105-agent:fix/remove-stale-domain-links
Aug 10, 2026
Merged

fix(sites): remove stale domain links on sync#110
ralyodio merged 1 commit into
profullstack:mainfrom
rissrice2105-agent:fix/remove-stale-domain-links

Conversation

@rissrice2105-agent

Copy link
Copy Markdown
Contributor

Summary

  • treat the domain database as authoritative during startup sync
  • remove entries from the domain symlink farm when they are absent from AllDomains()
  • rebuild every current mapping after stale entries are removed
  • add a regression for a stale domain-directory entry

Closes #109

Testing

  • go test ./internal/sites -run TestSyncRemovesStaleDomainEntries -count=1
  • go vet ./internal/sites
  • Linux internal/sites test binary compilation
  • gofmt verification
  • git diff --check

The full Windows package run reaches the repository's existing symlink integration test but cannot create symlinks without the Windows privilege; the new regression and Linux compilation both pass.

@ralyodio
ralyodio merged commit bf777a1 into profullstack:main Aug 10, 2026
6 checks passed
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.

Domain sync leaves stale site links after mappings are removed

2 participants