fix(#5465): discover mint URL from org variable for per-org repos

[fullsend-ai-coder]

Summary

fullsend repos init fails to discover the mint URL for per-org installations when orgCfg.DispatchSettings().MintURL is empty. This adds a fallback to check the org-level GitHub Actions variable FULLSEND_MINT_URL (set during fullsend admin install).

Related Issue

Fixes #5465

Changes

• Add org-level variable fallback in discoverRepo() (internal/repos/init.go): after checking orgCfg.DispatchSettings().MintURL, fall back to client.GetOrgVariable(ctx, owner, "FULLSEND_MINT_URL") when the mint URL is still empty
• Org config remains the primary source; the variable is best-effort
• GetOrgVariable errors are logged as warnings, not fatal
• Add three test cases covering: fallback discovery, org config precedence, and error handling

Testing

• All existing internal/repos tests pass
• Three new tests added:
  • TestDiscoverRepo_PerOrg_OrgVarFallback — verifies mint URL is discovered from org variable when org config has no mint_url
  • TestDiscoverRepo_PerOrg_OrgConfigTakesPrecedence — verifies org config mint_url wins over org variable
  • TestDiscoverRepo_PerOrg_OrgVarError_NonFatal — verifies GetOrgVariable errors produce a warning, not a failure
• go vet passes

Checklist

• PR title follows Conventional Commits (correct type, ! for breaking changes)
• Commits are signed off (DCO) — human and human-directed agent sessions only
• I wrote this contribution myself and can explain all changes in it

---

Closes #5465

Post-script verification

• Branch is not main/master (agent/5465-init-mint-url-org-var)
• Secret scan passed (gitleaks — 8b2146e80c0c8d8dc54fa6fa2b4f5957fc48e602..HEAD)
• PR body secret scan passed (gitleaks — no-git)
• Pre-commit hooks passed (authoritative run on runner)
• Tests ran inside sandbox

[fullsend-ai-review]

🤖 Review · ❌ Terminated · Started 3:40 PM UTC · Ended 4:03 PM UTC
Commit: bf2d784 · View workflow run →

[github-actions]

Site preview

Preview: https://b8a8ad58-site.fullsend-ai.workers.dev

Commit: cbee5349a9ca8c734d3c6936f620c14f58cfd855

[fullsend-ai-review]

🤖 Finished Review · ❌ Failure · Started 3:40 PM UTC · Completed 4:03 PM UTC
Commit: bf2d784 · View workflow run →

[ggallen]

/fs-review

[fullsend-ai-review]

🤖 Finished Review · ✅ Success · Started 4:09 PM UTC · Completed 4:23 PM UTC
Commit: 7ada4e0 · View workflow run →

[fullsend-ai-review]

Review

Findings

Medium

• [missing-documentation] docs/plans/repos-init.md:169 — The repos init plan documents per-org discovery as "Use mint URL and config from per-org config.yaml" but does not mention the new fallback to the org-level FULLSEND_MINT_URL variable when orgCfg.DispatchSettings().MintURL is empty.
  Remediation: Add a bullet noting the org variable fallback: "If no mint URL is set in config.yaml, fall back to reading the FULLSEND_MINT_URL org-level variable via GetOrgVariable."

Low

• [error-handling-idiom] internal/repos/init.go:370 — The error handling uses an else if err != nil chain with a compound condition declaring three variables inline, which differs from the separate-declaration pattern used for GetOrgVariable calls elsewhere. The current code is functionally correct — minor style nit.

• [missing-documentation] docs/cli/repos.md:52 — The repos init Discovery section lists three high-level checks but omits the org-level variable fallback for FULLSEND_MINT_URL. Low priority given the section's abstraction level.

---

Labels: PR modifies internal/repos/ (repos init command — part of the install workflow)

[fullsend-ai-review]

See the review comment for full details.

[ggallen]

/fs-fix. Rebase and fix all the review issues, both inline and in the sticky comment. Make sure to resolve all the conversations.

[ggallen]

/fs-fix - Rebase and fix all the review issues, both inline and in the sticky comment. Make sure to resolve all the conversations.

[fullsend-ai-coder]

🤖 Finished Fix · ✅ Success · Started 4:50 PM UTC · Completed 4:56 PM UTC
Commit: 7ada4e0 · View workflow run →

[fullsend-ai-coder]

🔧 Fix agent — iteration 1 (human-triggered)

Addressed all 3 review findings: refactored error handling idiom to match codebase convention, and added org variable fallback documentation to both the repos init plan and CLI docs.

Fixed (3):

1. error-handling-idiom: GetOrgVariable compound condition differs from separate-declaration pattern (internal/repos/init.go): Refactored the GetOrgVariable call in discoverRepo to use separate variable declaration and sequential error/exists checks, matching the idiom used in internal/cli/foreign.go and internal/mintcore/github.go
2. missing-documentation: repos init plan omits org variable fallback (docs/plans/repos-init.md): Added a note to the per-org discovery bullet explaining the FULLSEND_MINT_URL org-level variable fallback when config.yaml has no mint URL
3. missing-documentation: repos init CLI docs omit org variable fallback (docs/cli/repos.md): Added org variable fallback detail to the per-org config enrollment item in the Discovery section

Tests: passed

_{Updated by fullsend fix agent}