Add debt-ops (bcanfield/agentic-tech-debt)#194
Conversation
There was a problem hiding this comment.
Code Review
This pull request registers and implements the new debt-ops (agentic-tech-debt) Codex plugin, adding its configuration, hooks (drop.py, feedback.py, session-start.py, stop.py), and skills (register.py, review.py). Feedback on the changes highlights two issues in feedback.py: first, the $CHANGED_FILES expansion needs to split file paths into separate arguments to prevent subprocess failures when running tools without a shell; second, the test count warning mechanism should automatically initialize the test-count file if it is missing to ensure self-healing robustness.
Important
The consumer version of Gemini Code Assist on GitHub is being sunset. Starting June 18, 2026, new organization installations will be blocked, and all code review activity will officially cease on July 17, 2026.
For more details on the timeline and next steps, please review the Help Documentation.
|
Both review points were real and are now fixed upstream (released as v0.9.1); the mirror in this PR is refreshed to match.
Upstream commit: bcanfield/agentic-tech-debt@666e26d |
|
The plugin bundle and index entries are complete, but the |
|
I double-checked this one and I believe the install_url is correct as-is:
Happy to change it if you'd rather point install_url at the mirrored bundle, but that would be a generator-level convention change affecting all subdir plugins rather than a fix in this PR. |
Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
|
Before this PR can be merged, your plugin repo needs the HOL AI Plugin Scanner running in CI. This is a mandatory requirement for all submissions. Add this workflow to your plugin repo at name: HOL Plugin Scanner
on:
push:
branches: [main, master]
pull_request:
branches: [main, master]
permissions:
contents: read
security-events: write
jobs:
scan:
runs-on: ubuntu-latest
steps:
- uses: actions/checkout@11bd71901bbe5b1630ceea73d27597364c9af683 # v4.2.2
- name: HOL Plugin Scanner
uses: hashgraph-online/ai-plugin-scanner-action@v1
with:
plugin_dir: "."
mode: scan
min_score: 80
fail_on_severity: high
format: sarif
upload_sarif: trueAlso run the scanner locally and include the score in your PR description: pipx install plugin-scanner
plugin-scanner scan . --format textYour plugin needs a score of 80/130 or higher with no critical or high severity findings. Link the CI run or paste the score in this PR description. See the full guide: SCANNER_GUIDE.md Additional issues: |
Resolves the plugins.json header conflict: total recomputed from the merged array (107 = upstream's 106 + debt-ops), last_updated bumped to today. The plugins array, README, and marketplace.json auto-merged. debt-ops validates clean against upstream/main (1 changed dir, pass). Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
|
The HOL Plugin Scanner is now wired into CI on the source repo and passing.
|
Adds debt-ops under Development & Workflow: it catches tech debt as the agent writes it. Hooks watch each edit and log deferred decisions (loosened types, swallowed errors, "for now" defaults) to a registry in the repo; a review skill ranks paydown by file churn. Stdlib Python only, fully local, no network calls. MIT.
What's in the PR:
plugins/bcanfield/agentic-tech-debt/(generated withmirror_plugin_bundlefromscripts/generate_plugins_json.py, plugin rootcodex/in the source repo)EXTRA_MIRROR_PATHSentry forhooks/: the manifest points athooks/hooks.json, but the hook commands invoke sibling Python scripts in that directory at runtime, so the mirror needs the whole folderplugins.jsonand.agents/plugins/marketplace.jsonentries spliced at the README positionRan
check-alphabetical.pyandvalidate-plugin-pr.pylocally; both pass (one warning: no.codexignoreupstream yet).Smoke test:
codex plugin marketplace add bcanfield/agentic-tech-debtthen installdebt-ops.