Reduce cyclomatic complexity of tui() by extracting startup helpers #12

Closed

coding-agent-marvin8 wants to merge 3 commits from refs/pull/12/head into main

pull from: refs/pull/12/head

merge into: marvin8:main

marvin8:main

marvin8:pages

Conversation 1 Commits 3 Files changed 6 +279 -267

coding-agent-marvin8 commented 2026-07-10 07:11:09 +00:00 (Migrated from codeberg.org)

Copy link

Extracts two module-level helpers from tui() to bring its complexipy score from 20 down to 1.

_ensure_initial_credentials() handles .envrc migration, reading credentials, and the initial OAuth flow.
_maybe_renew_token() handles the token-expiry check and opportunistic renewal.

Closes #11

Extracts two module-level helpers from tui() to bring its complexipy score from 20 down to 1. _ensure_initial_credentials() handles .envrc migration, reading credentials, and the initial OAuth flow. _maybe_renew_token() handles the token-expiry check and opportunistic renewal. Closes #11

coding-agent-marvin8 commented 2026-07-10 07:37:19 +00:00 (Migrated from codeberg.org)

Copy link

Changes cherry-picked into PR #10 (feat/issue-8-9-error-handling-config) as commits 607142b, 36667ce, and bc58987. Closing this PR.

Changes cherry-picked into PR #10 (feat/issue-8-9-error-handling-config) as commits 607142b, 36667ce, and bc58987. Closing this PR.

Pull request closed

This pull request cannot be reopened because the branch was deleted.

Sign in to join this conversation.

Reviewers

No reviewers

Labels

Clear labels

No items

No labels

Milestone

Clear milestone

No items

No milestone

Assignees

Clear assignees

No assignees

1 participant

Notifications

Subscribe

Due date

The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Reference

marvin8/kokopelli!12

No description provided.