Skip to content

Fix archaeology workflow Python cache setup#35

Merged
simongonzalezdc merged 1 commit into
mainfrom
fix/archaeology-setup-python-cache
May 24, 2026
Merged

Fix archaeology workflow Python cache setup#35
simongonzalezdc merged 1 commit into
mainfrom
fix/archaeology-setup-python-cache

Conversation

@simongonzalezdc
Copy link
Copy Markdown
Member

@simongonzalezdc simongonzalezdc commented May 24, 2026

Summary

  • points the scheduled archaeology workflow's actions/setup-python pip cache at setup.py, matching this repo's legacy package shape
  • quotes $GITHUB_ENV in the project-name bootstrap step so the workflow passes actionlint cleanly

Why

The latest default-branch DevArch Archaeology Report failed before DevArch ran because setup-python could not find its default pip cache dependency files (requirements.txt or pyproject.toml). This repo currently has setup.py, so the cache dependency path needs to be explicit.

Fixes #34.

Verification

  • actionlint .github/workflows/archaeology.yml
  • git diff --check
  • static assertion that setup.py exists and .github/workflows/archaeology.yml references it as cache-dependency-path

View with Codesmith Autofix with Codesmith
Need help on this PR? Tag @codesmith with what you need. Autofix is disabled.

Summary by CodeRabbit

  • Chores
    • Improved CI/CD pipeline configuration for enhanced build efficiency and reliability.

Review Change Stack

The scheduled archaeology job failed before DevArch ran because setup-python tried to discover pip dependency files that this legacy setup.py package does not have. Pointing the cache at setup.py keeps the cache explicit and the job unblocked.

Tested: actionlint .github/workflows/archaeology.yml

Tested: git diff --check

Tested: static assertion that setup.py exists and cache-dependency-path references it
@coderabbitai
Copy link
Copy Markdown

coderabbitai Bot commented May 24, 2026

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro Plus

Run ID: 6fc1992f-1c44-4734-b1ee-bfb6d932224c

📥 Commits

Reviewing files that changed from the base of the PR and between 5c0d20a and def9f36.

📒 Files selected for processing (1)
  • .github/workflows/archaeology.yml

📝 Walkthrough

Walkthrough

Two targeted updates to the DevArch Archaeology Report workflow: the Python setup step now explicitly declares setup.py as a cache-dependency path to invalidate the Python cache when dependencies change, and the PROJECT_NAME environment variable export uses quoted syntax for the GitHub environment file reference.

Changes

GitHub Actions Workflow Configuration Fix

Layer / File(s) Summary
Dependency caching and environment variable handling
.github/workflows/archaeology.yml
Python setup step adds cache-dependency-path: setup.py to enable cache invalidation when dependencies change. PROJECT_NAME export to $GITHUB_ENV now uses quoted variable reference for correct shell interpretation.

🎯 1 (Trivial) | ⏱️ ~2 minutes

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Title check ✅ Passed The title accurately summarizes the main change: fixing the archaeology workflow's Python cache setup by pointing it at setup.py.
Description check ✅ Passed The description includes a clear summary, explains the root cause, references the linked issue #34, and lists verification steps, but does not address the Empower Orchestrator checklist template.
Linked Issues check ✅ Passed The PR directly addresses the root cause identified in issue #34 by pointing the cache-dependency-path at setup.py and fixing the actionlint warning, fulfilling the acceptance criteria.
Out of Scope Changes check ✅ Passed All changes are scoped to fixing the archaeology workflow failure: adding cache-dependency-path to setup-python and quoting $GITHUB_ENV in the bootstrap step.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch fix/archaeology-setup-python-cache

Comment @coderabbitai help to get the list of available commands and usage tips.

@simongonzalezdc simongonzalezdc merged commit c4ebdf1 into main May 24, 2026
12 checks passed
@simongonzalezdc simongonzalezdc deleted the fix/archaeology-setup-python-cache branch May 24, 2026 06:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[pipeline][HIGH] DevArch Archaeology Report fails on default branch

1 participant