feat: mount canonical gitea-workflow skill for Codex and MCP inventory (Closes #551) #565

Merged
sysadmin merged 4 commits from feat/issue-551-codex-gitea-workflow into master 2026-07-09 08:14:31 -05:00
Owner

Summary

Closes #551.

Makes the workflow skill wall resolve under the same canonical names across Claude, Codex, and MCP inventory so controller prompts requiring gitea-workflow no longer dead-end on Codex.

Changes

  • workflow_skill.py — canonical names (gitea-workflow, llm-project-workflow, git-pr-workflows), repo presence + Codex mount assessment
  • mcp_list_project_skills / mcp_get_skill_guide register all three names
  • New mcp_check_workflow_skill_preflight (fail closed if in-repo skill missing)
  • skills/gitea-workflow/SKILL.md alias stub → portable package
  • scripts/install-codex-workflow-skill.sh symlinks into ~/.codex/skills
  • Docs: docs/workflow-skill-mount.md + runbook section
  • Tests: tests/test_workflow_skill.py (+ operator guide expected skills)

Operator note

After merge, Codex hosts should run:

./scripts/install-codex-workflow-skill.sh

then restart Codex.

Worktree only

branches/feat-issue-551-codex-gitea-workflow @ 44cf2a4

Tests

pytest tests/test_workflow_skill.py tests/test_operator_guide.py -q
# 32 passed

Self-review: no (author = jcwalker3).

## Summary Closes #551. Makes the workflow skill wall resolve under the **same canonical names** across Claude, Codex, and MCP inventory so controller prompts requiring `gitea-workflow` no longer dead-end on Codex. ## Changes - `workflow_skill.py` — canonical names (`gitea-workflow`, `llm-project-workflow`, `git-pr-workflows`), repo presence + Codex mount assessment - `mcp_list_project_skills` / `mcp_get_skill_guide` register all three names - New `mcp_check_workflow_skill_preflight` (fail closed if in-repo skill missing) - `skills/gitea-workflow/SKILL.md` alias stub → portable package - `scripts/install-codex-workflow-skill.sh` symlinks into `~/.codex/skills` - Docs: `docs/workflow-skill-mount.md` + runbook section - Tests: `tests/test_workflow_skill.py` (+ operator guide expected skills) ## Operator note After merge, Codex hosts should run: ```bash ./scripts/install-codex-workflow-skill.sh ``` then restart Codex. ## Worktree only `branches/feat-issue-551-codex-gitea-workflow` @ `44cf2a4` ## Tests ``` pytest tests/test_workflow_skill.py tests/test_operator_guide.py -q # 32 passed ``` Self-review: no (author = jcwalker3).
jcwalker3 added 1 commit 2026-07-08 21:44:02 -05:00
Expose gitea-workflow / llm-project-workflow / git-pr-workflows as one
workflow router in mcp_list_project_skills, add preflight + Codex install
script, and document multi-runtime skill name parity.
jcwalker3 added 1 commit 2026-07-09 07:28:00 -05:00
jcwalker3 added 1 commit 2026-07-09 07:45:26 -05:00
jcwalker3 added 1 commit 2026-07-09 08:13:05 -05:00
Owner

repo: Scaled-Tech-Consulting/Gitea-Tools
pr: #565
issue: #551
reviewer_identity: sysadmin
profile: prgs-reviewer
session_id: 44998-e507e446ef05
worktree: /Users/jasonwalker/Development/Gitea-Tools/branches/review-pr-565-codex-gitea-workflow
phase: claimed
candidate_head: e8f93e93d7
target_branch: master
target_branch_sha: b52fa2060f
last_activity: 2026-07-09T13:13:47Z
expires_at: 2026-07-09T15:13:47Z
blocker: none

<!-- mcp-review-lease:v1 --> repo: Scaled-Tech-Consulting/Gitea-Tools pr: #565 issue: #551 reviewer_identity: sysadmin profile: prgs-reviewer session_id: 44998-e507e446ef05 worktree: /Users/jasonwalker/Development/Gitea-Tools/branches/review-pr-565-codex-gitea-workflow phase: claimed candidate_head: e8f93e93d70d5f0a486375e0c691a66ccd052639 target_branch: master target_branch_sha: b52fa2060fb3ae7fa1e90c1fc35cf0638b94c6b4 last_activity: 2026-07-09T13:13:47Z expires_at: 2026-07-09T15:13:47Z blocker: none
sysadmin approved these changes 2026-07-09 08:14:03 -05:00
sysadmin left a comment
Owner

Review: APPROVE PR #565

Closes #551 — mount canonical gitea-workflow skill for Codex and MCP inventory.

Pre-review note

PR head was rebased onto current master (conflict was docs-only in llm-workflow-runbooks.md). New head after push: e8f93e93d70d5f0a486375e0c691a66ccd052639.

Scope (9 files)

  • workflow_skill.py — canonical names, repo presence, Codex mount assessment
  • MCP: inventory + mcp_check_workflow_skill_preflight
  • skills/gitea-workflow/SKILL.md alias stub
  • scripts/install-codex-workflow-skill.sh + docs/tests

Validation

  • py_compile clean
  • Focused tests: 33 passed
  • Mergeable against current master after rebase

Assessment

  • Solves controller/Codex name dead-end for gitea-workflow
  • Fail-closed preflight when in-repo skill missing
  • Operator install path is explicit and dry-run capable

Identity

  • Reviewer: sysadmin / prgs-reviewer
  • Author: jcwalker3 (not self-review)

Canonical PR State

STATE: approved ready-to-merge
WHO_IS_NEXT: merger
NEXT_ACTION: merge PR #565 at the pinned head with expected_changed_files
NEXT_PROMPT:

Merge PR #565 (confirmation MERGE PR 565) at head e8f93e93d70d5f0a486375e0c691a66ccd052639 using prgs-merger. Files: docs/llm-workflow-runbooks.md, docs/workflow-skill-mount.md, gitea_mcp_server.py, scripts/install-codex-workflow-skill.sh, skills/gitea-workflow/SKILL.md, skills/llm-project-workflow/SKILL.md, tests/test_operator_guide.py, tests/test_workflow_skill.py, workflow_skill.py. Then continue web UI stack (#452–#457).

WHAT_HAPPENED: Rebased onto master (doc conflict resolved), re-validated, formal APPROVE.
WHY: Scope matches #551; tests green; mergeable at rebased head.
ISSUE: #551
HEAD_SHA: e8f93e93d7
REVIEW_STATUS: approved
MERGE_READY: yes
BLOCKERS: none
VALIDATION: 33 focused tests passed; mergeable True on current master
LAST_UPDATED_BY: sysadmin / prgs-reviewer

## Review: APPROVE PR #565 **Closes #551** — mount canonical gitea-workflow skill for Codex and MCP inventory. ### Pre-review note PR head was rebased onto current master (conflict was docs-only in `llm-workflow-runbooks.md`). New head after push: `e8f93e93d70d5f0a486375e0c691a66ccd052639`. ### Scope (9 files) - `workflow_skill.py` — canonical names, repo presence, Codex mount assessment - MCP: inventory + `mcp_check_workflow_skill_preflight` - `skills/gitea-workflow/SKILL.md` alias stub - `scripts/install-codex-workflow-skill.sh` + docs/tests ### Validation - py_compile clean - Focused tests: **33 passed** - Mergeable against current master after rebase ### Assessment - Solves controller/Codex name dead-end for `gitea-workflow` - Fail-closed preflight when in-repo skill missing - Operator install path is explicit and dry-run capable ### Identity - Reviewer: sysadmin / prgs-reviewer - Author: jcwalker3 (not self-review) ## Canonical PR State STATE: approved ready-to-merge WHO_IS_NEXT: merger NEXT_ACTION: merge PR #565 at the pinned head with expected_changed_files NEXT_PROMPT: ```text Merge PR #565 (confirmation MERGE PR 565) at head e8f93e93d70d5f0a486375e0c691a66ccd052639 using prgs-merger. Files: docs/llm-workflow-runbooks.md, docs/workflow-skill-mount.md, gitea_mcp_server.py, scripts/install-codex-workflow-skill.sh, skills/gitea-workflow/SKILL.md, skills/llm-project-workflow/SKILL.md, tests/test_operator_guide.py, tests/test_workflow_skill.py, workflow_skill.py. Then continue web UI stack (#452–#457). ``` WHAT_HAPPENED: Rebased onto master (doc conflict resolved), re-validated, formal APPROVE. WHY: Scope matches #551; tests green; mergeable at rebased head. ISSUE: #551 HEAD_SHA: e8f93e93d70d5f0a486375e0c691a66ccd052639 REVIEW_STATUS: approved MERGE_READY: yes BLOCKERS: none VALIDATION: 33 focused tests passed; mergeable True on current master LAST_UPDATED_BY: sysadmin / prgs-reviewer
Owner

adopted_at: 2026-07-09T13:14:18Z
adopted_by_identity: sysadmin
adopted_by_profile: prgs-merger
adopted_from_session_id: 44998-e507e446ef05
adopted_from_profile: prgs-reviewer
adopted_from_reviewer_identity: sysadmin
adopted_from_comment_id: 7965
adoption_reason: merger-handoff-approved-head

repo: Scaled-Tech-Consulting/Gitea-Tools
pr: #565
issue: #551
reviewer_identity: sysadmin
profile: prgs-merger
session_id: 45883-8e58d26be3e4
worktree: /Users/jasonwalker/Development/Gitea-Tools/branches/review-pr-565-codex-gitea-workflow
phase: adopted
candidate_head: e8f93e93d7
target_branch: master
target_branch_sha: b52fa2060f
last_activity: 2026-07-09T13:14:18Z
expires_at: 2026-07-09T15:14:18Z
blocker: none

<!-- mcp-review-lease-adoption:v1 --> adopted_at: 2026-07-09T13:14:18Z adopted_by_identity: sysadmin adopted_by_profile: prgs-merger adopted_from_session_id: 44998-e507e446ef05 adopted_from_profile: prgs-reviewer adopted_from_reviewer_identity: sysadmin adopted_from_comment_id: 7965 adoption_reason: merger-handoff-approved-head <!-- mcp-review-lease:v1 --> repo: Scaled-Tech-Consulting/Gitea-Tools pr: #565 issue: #551 reviewer_identity: sysadmin profile: prgs-merger session_id: 45883-8e58d26be3e4 worktree: /Users/jasonwalker/Development/Gitea-Tools/branches/review-pr-565-codex-gitea-workflow phase: adopted candidate_head: e8f93e93d70d5f0a486375e0c691a66ccd052639 target_branch: master target_branch_sha: b52fa2060fb3ae7fa1e90c1fc35cf0638b94c6b4 last_activity: 2026-07-09T13:14:18Z expires_at: 2026-07-09T15:14:18Z blocker: none
sysadmin merged commit 254ccf5821 into master 2026-07-09 08:14:31 -05:00
Sign in to join this conversation.
No Reviewers
No labels
2 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: Scaled-Tech-Consulting/Gitea-Tools#565