[PR #215] [MERGED] Fix preserve-mode fork owner namespace collisions #1088

Closed
opened 2026-04-19 13:39:22 -05:00 by GiteaMirror · 0 comments
Owner

📋 Pull Request Information

Original PR: https://github.com/RayLabsHQ/gitea-mirror/pull/215
Author: @arunavo4
Created: 3/6/2026
Status: Merged
Merged: 3/6/2026
Merged by: @arunavo4

Base: mainHead: codex/issue-214-preserve-fork-owner


📝 Commits (1)

  • 4d23f47 fix preserve strategy fork owner routing

📊 Changes

2 files changed (+95 additions, -4 deletions)

View changed files

📝 src/lib/gitea.test.ts (+48 -2)
📝 src/lib/gitea.ts (+47 -2)

📄 Description

Summary

  • fix preserve strategy owner resolution for personal repos owned by other users
  • ensure preserve mode does not fall back to defaultOwner when external owner namespace creation fails (fail-safe to avoid cross-owner overwrite)
  • add regression coverage for external owner preserve behavior

Testing

  • bun test src/lib/gitea.test.ts

Closes #214


🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.

## 📋 Pull Request Information **Original PR:** https://github.com/RayLabsHQ/gitea-mirror/pull/215 **Author:** [@arunavo4](https://github.com/arunavo4) **Created:** 3/6/2026 **Status:** ✅ Merged **Merged:** 3/6/2026 **Merged by:** [@arunavo4](https://github.com/arunavo4) **Base:** `main` ← **Head:** `codex/issue-214-preserve-fork-owner` --- ### 📝 Commits (1) - [`4d23f47`](https://github.com/RayLabsHQ/gitea-mirror/commit/4d23f4717b38757b9259306d0c91bc165c375712) fix preserve strategy fork owner routing ### 📊 Changes **2 files changed** (+95 additions, -4 deletions) <details> <summary>View changed files</summary> 📝 `src/lib/gitea.test.ts` (+48 -2) 📝 `src/lib/gitea.ts` (+47 -2) </details> ### 📄 Description ## Summary - fix preserve strategy owner resolution for personal repos owned by other users - ensure preserve mode does not fall back to defaultOwner when external owner namespace creation fails (fail-safe to avoid cross-owner overwrite) - add regression coverage for external owner preserve behavior ## Testing - bun test src/lib/gitea.test.ts Closes #214 --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>
GiteaMirror added the pull-request label 2026-04-19 13:39:22 -05:00
Sign in to join this conversation.
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: github-starred/gitea-mirror#1088