fix: tech-debt batch 4 — parseIssueHref, job limit, stale TODOs #360

2026-05-31T16:13:24Z

jmiller commented

2026-05-31 16:13:24 +00:00

Summary

parseIssueHref fix: Now uses URL to extract pathname and trims appSubUrl before matching, fixing issue link parsing for sub-path deployments (e.g. /gitea/owner/repo/issues/1)
Actions job limit enforcement: Workflows with >256 jobs now fail fast with a clear error message instead of silently proceeding
Remove stale TODO: OAuth authorize route comment was not actionable

Refs #325, #334

Test plan

Verify issue links parse correctly in sub-path deployments
Verify workflows with <=256 jobs still work
Verify workflows with >256 jobs return an error

🤖 Generated with Claude Code

## Summary - **`parseIssueHref` fix**: Now uses `URL` to extract pathname and trims `appSubUrl` before matching, fixing issue link parsing for sub-path deployments (e.g. `/gitea/owner/repo/issues/1`) - **Actions job limit enforcement**: Workflows with >256 jobs now fail fast with a clear error message instead of silently proceeding - **Remove stale TODO**: OAuth authorize route comment was not actionable Refs #325, #334 ## Test plan - [ ] Verify issue links parse correctly in sub-path deployments - [ ] Verify workflows with <=256 jobs still work - [ ] Verify workflows with >256 jobs return an error 🤖 Generated with [Claude Code](https://claude.com/claude-code)

jmiller added 1 commit 2026-05-31 16:13:25 +00:00

fix: resolve tech-debt batch 4 — parseIssueHref, job limit, stale TODOs

Universal: PR Check / Branch Policy (pull_request) Successful in 2s

Details

Branch Policy Check / Verify merge target (pull_request) Successful in 2s

Details

PR RC Release / Build RC Release (pull_request) Successful in 3s

Details

Universal: PR Check / Validate PR (pull_request) Failing after 8s

Details

Branch Cleanup / Delete merged branch (pull_request) Successful in 2s

Details

Universal: PR Check / Build RC Package (pull_request) Has been cancelled

Details

889f64009b

- fix(ts): parseIssueHref now uses URL pathname and trims appSubUrl
  for correct issue link parsing with sub-path deployments
- fix(actions): enforce MaxJobNumPerRun (256) limit when creating jobs,
  rejecting workflows that exceed the GitHub-compatible limit
- chore: remove stale TODO comment on OAuth redirect route

Refs #325, #334

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>

jmiller merged commit c1c7556916 into dev

2026-05-31 16:13:39 +00:00

jmiller deleted branch fix/tech-debt-batch-4

2026-05-31 16:13:39 +00:00

Sign in to join this conversation.

1 Participants

Notifications

Due Date

No due date set.

Dependencies

No dependencies set.

Reference: MokoConsulting/MokoGitea#360