Full framework overhaul: 100% parity + open-source packaging#3
Conversation
…ce packaging Complete archaeology/ package port (25 modules) with all CLI commands, analysis vectors, visualizations, and pipeline integration. Includes: - setup.py for pip-installable CLI (devarch command) - Demo project with 6 commits, 2 sessions, 3 eras - All 6 analysis vectors with sterilized templates - Agent benchmark + multi-project dashboard visualizations - Git hooks (pre-commit era scanner, pre-push audit+parity) - CI workflow (Python 3.10/3.11/3.12 matrix) - Open-source packaging: CONTRIBUTING.md, CHANGELOG.md, Makefile - GitHub issue templates + config - config/profile.json template for multi-project sync Generalized all "liminal" references to "primary" for framework use. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
![chatgpt-codex-connector[bot]](https://avatars.githubusercontent.com/in/1144995?s=60&v=4){kind=small}
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 6126afbcf0
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
| stale_s = fmt_int(stale) | ||
| # "N commits" or "N," (with comma formatting variants) | ||
| slots.append(( | ||
| re.compile(rf"\b{stale_s.replace(',', r'[,.]?\s*')}?\s*{stale}\s+commits\b"), |
There was a problem hiding this comment.
The regex on this line makes the module fail to import at all: Python raises SyntaxError: f-string expression part cannot include a backslash, so scripts/sync/sync_derived_deliverables.py cannot run in either sync or check mode. Because this is a hard parse-time failure, any workflow invoking the script will break before doing any work.
Useful? React with 👍 / 👎.
| run([sys.executable, "pipeline/core/validate.py", "--strict"]) | ||
| run([sys.executable, "pipeline/core/run.py", "--validate"]) |
There was a problem hiding this comment.
The regeneration flow calls pipeline/core/validate.py and pipeline/core/run.py, but this repository no longer contains a pipeline/ tree, so the default validate() phase will fail with missing-file errors and abort regeneration. Since main() runs validate() unless --skip-validate is passed, this breaks the script’s standard execution path.
Useful? React with 👍 / 👎.
Old test imported from pipeline.core.validate which no longer exists. Rewritten to test archaeology.audit.AuditFinding with correct API. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Framework only has demo-project, not liminal. Updated audit tests to test against the actual project that exists. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
…._as_int Three tests imported MetricValidator from pipeline.core.validate which no longer exists. Replaced with equivalent tests for audit._as_int(). Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
- Replace Path.rename() with os.replace() in atomic_write for cross-platform atomic replacement (Windows raises FileExistsError) - Skip chmod-based IO test on Windows (Unix permissions ignored) Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
1 participant
Summary
devarchcommand)Test plan
python3 -c "import archaeology")🤖 Generated with Claude Code
Need help on this PR? Tag
@codesmithwith what you need.