laantungir/ngit-grasp
1
0
Fork 0
mirror of https://relay.ngit.dev/npub15qydau2hjma6ngxkl2cyar74wzyjshvl65za5k5rl69264ar2exs5cyejr/ngit-grasp.git synced 2026-10-05 15:08:24 +00:00
Code Issues Packages Projects Releases Wiki Activity

issue: complete 6af6 - archive_read_only bare repo fix ready to merge

Browse Source
This commit is contained in:
DanConwayDev
2026-01-19 14:57:24 +00:00
parent 4b34150310
commit 20cdd9acd3
1 changed files with 10 additions and 0 deletions
Download Patch File Download Diff File Expand all files Collapse all files
6af6-archive-read-only-should-create-bare-repos.md
+10
View File
@@ -145,6 +145,16 @@ When `archive_read_only = true` is set with `archive_all = true`, the relay acce
- Security model validated
- All phases complete
### 2026-01-19 [Session 20:00] - FINAL REVIEW AND MERGE
- **BUILD AGENT REVIEW**: Reviewed test implementation for quality and isolation
- Test isolation: ✅ Uses random temp directories and ports - no conflicts possible
- Test complexity: ✅ Bare repo check is necessary and justified - validates the actual fix
- Recommendation: Keep current implementation - well-designed and ready to merge
- **ISSUE COMPLETE**: All work committed, tests pass, ready to merge to main
- Commit bf70d72: "fix: archive_read_only creates bare repos for archived announcements"
- Test coverage comprehensive and properly isolated
- No changes needed - merging to main
## Notes
- Related code: `src/nostr/builder.rs:169-195` (AcceptArchive handling)