DEVX-123: Role-based Gitea API tokens #186

Merged
emil merged 5 commits from DEVX-123-role-based-gitea-tokens into master 2026-07-08 19:30:12 +00:00
Owner

Closes DEVX-123.

Introduces role-based Gitea API token environment variables:

  • CI_GITEA_API_TOKEN for CI workflows
  • REVIEWER_GITEA_API_TOKEN for PR approvals (different user from author)
  • DEVELOPER_GITEA_API_TOKEN for local tooling

Adds devx.tokens helpers with backwards compatibility for legacy names. All devx CI scripts, tools, workflows, Makefile, and .env.example updated to use the role-appropriate token.

Closes DEVX-123. Introduces role-based Gitea API token environment variables: - `CI_GITEA_API_TOKEN` for CI workflows - `REVIEWER_GITEA_API_TOKEN` for PR approvals (different user from author) - `DEVELOPER_GITEA_API_TOKEN` for local tooling Adds `devx.tokens` helpers with backwards compatibility for legacy names. All devx CI scripts, tools, workflows, Makefile, and `.env.example` updated to use the role-appropriate token.
emil added 1 commit 2026-07-08 18:56:28 +00:00
feat: introduce role-based Gitea API token environment variables
CI / pr-review (pull_request) Failing after 8s
CI / detect-changes (pull_request) Successful in 16s
CI / quality (pull_request) Failing after 41s
CI / release-dry-run (pull_request) Has been skipped
CI / auto-merge (pull_request) Has been skipped
10bc1afd83
Add `devx.tokens` helper module with `get_ci_token()`,
`get_reviewer_token()`, `get_developer_token()`, and `get_vikunja_token()`.
All devx tools, CI scripts, molecule helpers, workflows, Makefile, and
.env.example now use the role-appropriate token:
- `CI_GITEA_API_TOKEN` for CI workflows
- `REVIEWER_GITEA_API_TOKEN` for PR approvals (must be a different user
  from the PR author)
- `DEVELOPER_GITEA_API_TOKEN` for local development tooling

Legacy names `CI_GITEA_TOKEN` and `REVIEW_GITEA_TOKEN` remain accepted
for backwards compatibility. Update tests to cover the new helpers and
ensure 100% coverage.

DEVX-123

Generated with [Devin](https://devin.ai)

Co-Authored-By: Devin <158243242+devin-ai-integration[bot]@users.noreply.github.com>
emil added 1 commit 2026-07-08 19:10:18 +00:00
ci: retrigger workflow after configuring secrets
CI / pr-review (pull_request) Failing after 10s
CI / detect-changes (pull_request) Successful in 16s
CI / quality (pull_request) Failing after 35s
CI / release-dry-run (pull_request) Has been skipped
CI / auto-merge (pull_request) Has been skipped
3dfd194ef8
emil reviewed 2026-07-08 19:13:16 +00:00
emil left a comment
Author
Owner

test

test
emil added 1 commit 2026-07-08 19:16:13 +00:00
ci: retrigger after updating Gitea Actions secrets
CI / pr-review (pull_request) Failing after 10s
CI / detect-changes (pull_request) Successful in 14s
CI / quality (pull_request) Failing after 44s
CI / release-dry-run (pull_request) Has been skipped
CI / auto-merge (pull_request) Has been skipped
e8626b0716
emil added 1 commit 2026-07-08 19:19:55 +00:00
ci: retrigger after setting secret values
CI / pr-review (pull_request) Successful in 13s
CI / detect-changes (pull_request) Successful in 17s
CI / quality (pull_request) Failing after 36s
CI / release-dry-run (pull_request) Has been skipped
CI / auto-merge (pull_request) Has been skipped
6a4b17632f
emil reviewed 2026-07-08 19:20:09 +00:00
emil left a comment
Author
Owner

Automated PR Review

  • Architecture compliance: OK
  • Best practices: OK
  • Security: OK
  • i18n: OK
  • Resource management: OK
  • Documentation: WARNING — source files changed but no docs updated
  • Tests: WARNING — source files changed but no test files updated
  • Commit conventions: OK

No issues found by automated checks.


Auto-merge: If all CI checks pass, this PR will be merged automatically.

## Automated PR Review - Architecture compliance: OK - Best practices: OK - Security: OK - i18n: OK - Resource management: OK - Documentation: WARNING — source files changed but no docs updated - Tests: WARNING — source files changed but no test files updated - Commit conventions: OK No issues found by automated checks. --- **Auto-merge:** If all CI checks pass, this PR will be merged automatically.
emil added 1 commit 2026-07-08 19:25:55 +00:00
fix: add missing token translation key and update token isolation tests
CI / pr-review (pull_request) Successful in 12s
CI / detect-changes (pull_request) Successful in 15s
CI / quality (pull_request) Successful in 46s
CI / release-dry-run (pull_request) Has been skipped
CI / auto-merge (pull_request) Successful in 15s
cd7ab0c08b
Add the translation key for the new  error message.
Update unit tests that assert 'no token' behaviour to also clear the
new role-based token environment variables so they remain isolated when
 or  is set in the
environment.

DEVX-123
emil reviewed 2026-07-08 19:26:05 +00:00
emil left a comment
Author
Owner

Automated PR Review

  • Architecture compliance: OK
  • Best practices: OK
  • Security: OK
  • i18n: OK
  • Resource management: OK
  • Documentation: WARNING — source files changed but no docs updated
  • Tests: WARNING — source files changed but no test files updated
  • Commit conventions: OK

No issues found by automated checks.


Auto-merge: If all CI checks pass, this PR will be merged automatically.

## Automated PR Review - Architecture compliance: OK - Best practices: OK - Security: OK - i18n: OK - Resource management: OK - Documentation: WARNING — source files changed but no docs updated - Tests: WARNING — source files changed but no test files updated - Commit conventions: OK No issues found by automated checks. --- **Auto-merge:** If all CI checks pass, this PR will be merged automatically.
kireto approved these changes 2026-07-08 19:27:01 +00:00
Dismissed
kireto left a comment
Member

Auto-approved: all CI checks passed (quality, pr-review, release-dry-run).

Auto-approved: all CI checks passed (quality, pr-review, release-dry-run).
emil reviewed 2026-07-08 19:29:04 +00:00
emil left a comment
Author
Owner

Automated PR Review

  • Architecture compliance: OK
  • Best practices: OK
  • Security: OK
  • i18n: OK
  • Resource management: OK
  • Documentation: WARNING — source files changed but no docs updated
  • Tests: WARNING — source files changed but no test files updated
  • Commit conventions: OK

No issues found by automated checks.


Auto-merge: If all CI checks pass, this PR will be merged automatically.

## Automated PR Review - Architecture compliance: OK - Best practices: OK - Security: OK - i18n: OK - Resource management: OK - Documentation: WARNING — source files changed but no docs updated - Tests: WARNING — source files changed but no test files updated - Commit conventions: OK No issues found by automated checks. --- **Auto-merge:** If all CI checks pass, this PR will be merged automatically.
kireto approved these changes 2026-07-08 19:30:08 +00:00
kireto left a comment
Member

Auto-approved: all CI checks passed (quality, pr-review, release-dry-run).

Auto-approved: all CI checks passed (quality, pr-review, release-dry-run).
emil merged commit 0228fce5b9 into master 2026-07-08 19:30:12 +00:00
emil deleted branch DEVX-123-role-based-gitea-tokens 2026-07-08 19:30:13 +00:00
Sign in to join this conversation.
No Reviewers
2 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: oblachno-oss/devx#186