Refactor (packages/opencode/src/util/repository.ts:139): Function with many returns (count = 8): parseRepositoryReference

[Wl1u4]

Closes #171

Closes #

Type of change

• Bug fix
• New feature
• [ Check] Refactor / code improvement
• Documentation

What does this PR do?

Fixes #171. Qlty flagged parseRepositoryReference in packages/opencode/src/util/repository.ts for having too many return statements (8). The func try several parsing strats in a row: GitHub shorthand like "owner/repo", "github:owner/repo" prefix, SCP-style git URLs, direct host/path, and full URLs. returning as soon as one matches.

I pulled each strategy out into its own small func (matchGithubPrefixed, matchScpStyle, matchDirectHostPath, matchGithubShorthand, matchUrl), put them in an array, and looped through them, returning the first non null result. This keeps exact same order and fallback behavior as before, but main func now only has 2 return statements instead of 8, and each strategy is easier to read/test on its own.

If you paste a large clearly AI generated description here your PR may be IGNORED or CLOSED!

How did you verify your code works?

Ran qlty smells before and after. the "many returns" warning is gone and no new smells were introduced.

Ran existing test suite (test/util/repository.test.ts) all 6 tests still pass with no changes needed.

Also ran the full test suite (bun test --coverage) to make sure nothing else broke: 3254 pass, 4 fail (those 4 failures are pre exisitng ones and unrelated to this file).

Checked coverage on the changed file specifically

and confirmed the few uncovered lines

Also ran bun run typecheck with no errors.

Checklist

• [ Check] I have tested my changes locally
• [Check ] I have not included unrelated changes in this PR

If you do not follow this template your PR will be automatically rejected.