Restore ArmorCodex entry in Community Plugins (lost during PR #115 reconcile)#189
Restore ArmorCodex entry in Community Plugins (lost during PR #115 reconcile)#189Harihara04sudhan wants to merge 6 commits into
Conversation
There was a problem hiding this comment.
Code Review
This pull request adds the ArmorCodex plugin to the list of third-party plugins in README.md. The review feedback suggests enclosing 'bash' and 'apply_patch' in backticks to maintain formatting consistency and prevent markdown parsing issues.
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.
| - [Agentgram](https://github.com/jerryfane/agentgram) - Send explicit Telegram messages from Codex and local AI agents through a Telegram bot token and chat id. | ||
| - [Aient](https://github.com/haf/aient-codex-plugin) - AI operations plugin for Codex that connects production telemetry, problem lifecycle context, and remediation workflows through Aient's MCP server. | ||
| - [Apple Productivity](https://github.com/matk0shub/apple-productivity-mcp) - Local Apple Calendar and Reminders tooling for macOS with Codex plugin adapters. | ||
| - [ArmorCodex](https://github.com/armoriq/armorCodex) - Intent-based security for Codex with MCP plan registration, policy gating, CSRG cryptographic proofs, and audit logging on Bash and apply_patch. |
There was a problem hiding this comment.
To maintain consistency with other entries in this document (such as rg, grep, and git blame), and to ensure that the underscore in apply_patch is not misinterpreted by markdown parsers, it is recommended to enclose bash (preferably lowercase) and apply_patch in backticks.\n\nFor example:\n- [ArmorCodex](https://github.com/armoriq/armorCodex) - Intent-based security for Codex with MCP plan registration, policy gating, CSRG cryptographic proofs, and audit logging on bashandapply_patch.
|
This PR only adds a README entry. For this bundle-required repo, a complete submission needs the full plugin bundle under |
|
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: |
07d5985 to
3ec8d05
Compare
Per @internet-dot's review comment requesting the full plugin bundle (not just the README entry). This commit adds the complete ArmorCodex plugin source tree, mirrored from armoriq/armorCodex@main: - README.md, LICENSE, SECURITY.md - .codexignore, .plugin-scanner.toml - hooks/hooks.json (8 hook events declared) - scripts/bootstrap.mjs, scripts/hook-router.mjs, scripts/policy-mcp.mjs - scripts/lib/ (13 lib modules: engine, intent, policy, audit-wal, etc.) - .agents/plugins/marketplace.json (per-plugin marketplace metadata, follows AgiFlow reference shape — name "armoriq", plugin name "armorcodex", category "Tools & Integrations", source URL pointing at the canonical github.com/armoriq/armorCodex repo) Excluded from the bundle: - node_modules/ (installed at runtime via package-lock.json) - tests/ (moved out of the plugin distribution dir per scanner compliance; lives at the repo root in the source repo now) Refs hashgraph-online#189
|
@internet-dot — all asks addressed. Single summary: What's now in the PR
What's excluded from the bundle (intentionally)
On the repo-wide
|
internet-dot
left a comment
There was a problem hiding this comment.
Two issues:
-
No scanner evidence — PR description lacks scanner score or CI link (mandatory).
-
Lowercase repo name in path — The bundle path is
plugins/armoriq/armorCodexwith mixed case. While not a blocker, the standard convention uses the repo's actual GitHub casing. Verify this matches the upstream repo structure.
Bundle otherwise looks complete: plugin.json, icon, SECURITY.md, LICENSE, marketplace.json, README entry. Alphabetical order is correct (Ar after Antigravity, before AxonFlow).
|
@internet-dot — addressed both review asks. 1. Scanner evidence in PR description — added. PR body now includes:
2. Lowercase repo name in path — renamed Ready for re-review. |
internet-dot
left a comment
There was a problem hiding this comment.
Review Summary
✅ Approved
Checks Passed:
- README.md: ArmorCodex entry restored alphabetically between 'Antigravity 2.0' and 'AxonFlow' ✓
- Bundle structure: Complete plugin bundle under with:
- ✓
- ✓
- ✓
- ✓
- (MIT) ✓
- ✓
- ✓
- ✓
- Complete scripts/ and hooks/ directories ✓
- marketplace.json: Per-plugin marketplace metadata added ✓
- Alphabetical order: Correctly positioned ✓
- No duplicates: Single entry ✓
Notes:
- Path lowercased from to per maintainer convention ✓
- Plugin scanner evidence provided (92/100 score, zero critical/high)
- Well-documented PR explaining the accidental deletion context
Thank you for the thorough restoration!
f1befbe to
25022e7
Compare
Per @internet-dot's review comment requesting the full plugin bundle (not just the README entry). This commit adds the complete ArmorCodex plugin source tree, mirrored from armoriq/armorCodex@main: - README.md, LICENSE, SECURITY.md - .codexignore, .plugin-scanner.toml - hooks/hooks.json (8 hook events declared) - scripts/bootstrap.mjs, scripts/hook-router.mjs, scripts/policy-mcp.mjs - scripts/lib/ (13 lib modules: engine, intent, policy, audit-wal, etc.) - .agents/plugins/marketplace.json (per-plugin marketplace metadata, follows AgiFlow reference shape — name "armoriq", plugin name "armorcodex", category "Tools & Integrations", source URL pointing at the canonical github.com/armoriq/armorCodex repo) Excluded from the bundle: - node_modules/ (installed at runtime via package-lock.json) - tests/ (moved out of the plugin distribution dir per scanner compliance; lives at the repo root in the source repo now) Refs hashgraph-online#189
|
@internet-dot — branch had a conflict (upstream main had already moved Agent Workflow System to the right alphabetical slot, our older commit was a duplicate). Just rebased onto main and resolved. PR is back to MERGEABLE. Could you please re-approve and merge whenever you have a moment? Thanks. |
|
Thanks for working on this. The plugin bundle looks good. Please also add:
The README entry can be handled once those registry files are updated. |
The ArmorCodex entry was added in PR hashgraph-online#140 (merged 2026-05-20 14:45 UTC) but accidentally dropped 5 minutes later when PR hashgraph-online#115 (Add 10 pluginpool plugins, merged 14:50 UTC) was reconciled. The plugin bundle at plugins/armoriq/armorCodex/ remained intact, so the registry has the plugin but README and downstream marketplace artifacts don't list it. Restoring the README entry in its alphabetical slot between Apple Productivity and AxonFlow. Plugin folder + plugin.json are already in the repo from PR hashgraph-online#140; no other changes needed. Repo: https://github.com/armoriq/armorCodex Plugin bundle in this repo: plugins/armoriq/armorCodex/
Per gemini-code-assist review on this PR — keeps formatting consistent with other entries (rg, grep, git blame) and prevents markdown parser ambiguity around the underscore in apply_patch.
Per @internet-dot's review comment requesting the full plugin bundle (not just the README entry). This commit adds the complete ArmorCodex plugin source tree, mirrored from armoriq/armorCodex@main: - README.md, LICENSE, SECURITY.md - .codexignore, .plugin-scanner.toml - hooks/hooks.json (8 hook events declared) - scripts/bootstrap.mjs, scripts/hook-router.mjs, scripts/policy-mcp.mjs - scripts/lib/ (13 lib modules: engine, intent, policy, audit-wal, etc.) - .agents/plugins/marketplace.json (per-plugin marketplace metadata, follows AgiFlow reference shape — name "armoriq", plugin name "armorcodex", category "Tools & Integrations", source URL pointing at the canonical github.com/armoriq/armorCodex repo) Excluded from the bundle: - node_modules/ (installed at runtime via package-lock.json) - tests/ (moved out of the plugin distribution dir per scanner compliance; lives at the repo root in the source repo now) Refs hashgraph-online#189
…rmorcodex Per @internet-dot review comment: convention used by most other plugins in this repo (ai-plugin, codex-mem, unity-agent-workflows, etc.) is lowercase for the repo segment of the bundle path. GitHub URLs in plugin.json and marketplace.json still reference the canonical github.com/armoriq/armorCodex (capital C preserved for the upstream repo). Only the bundle directory inside awesome-codex-plugins is lowercased.
Per @internet-dot's review comment (2026-06-16): the registry needed explicit entries in the top-level plugins.json and .agents/plugins/ marketplace.json (we already had the per-plugin marketplace.json inside the bundle from the previous commit). plugins.json entry: name: ArmorCodex, owner: armoriq, repo: armorCodex, category: Tools & Integrations, install_url: raw.githubusercontent.com/armoriq/armorCodex/HEAD/ .codex-plugin/plugin.json .agents/plugins/marketplace.json entry: name: armorcodex (lowercase, matches bundle dir), source: local ./plugins/armoriq/armorcodex, policy: installation AVAILABLE, authentication ON_INSTALL, category: Tools & Integrations. Bumped plugins.json "total" from 112 → 113.
25022e7 to
845478e
Compare
|
@internet-dot — added both registry entries in commit 1. {
"name": "ArmorCodex",
"url": "https://github.com/armoriq/armorCodex",
"owner": "armoriq",
"repo": "armorCodex",
"description": "Intent-based security for Codex with MCP plan registration, policy gating, CSRG cryptographic proofs, and audit logging on `bash` and `apply_patch`.",
"category": "Tools & Integrations",
"source": "awesome-codex-plugins",
"install_url": "https://raw.githubusercontent.com/armoriq/armorCodex/HEAD/.codex-plugin/plugin.json"
}Bumped 2. {
"name": "armorcodex",
"displayName": "ArmorCodex",
"source": {
"source": "local",
"path": "./plugins/armoriq/armorcodex"
},
"policy": {
"installation": "AVAILABLE",
"authentication": "ON_INSTALL"
},
"category": "Tools & Integrations",
"description": "Intent-based security for Codex with MCP plan registration, policy gating, CSRG cryptographic proofs, and audit logging on `bash` and `apply_patch`."
}Diff is clean (25 lines total, just the two entries — no formatting churn). Branch is rebased on latest main, MERGEABLE. Ready when you are. |
…Matter, Rust Reverse Engineering before ScrapeGraph AI Two pre-existing alphabetical mis-orderings surfaced by the alphabetical-check.py CI step on this PR. Neither is caused by the ArmorCodex addition (both lists already had ArmorCodex in the correct slot); fixing here so the gate clears. - Development & Workflow: "Generative Media Skills" (gen) was after "GrayMatter" (gra). Moved before. - Tools & Integrations: "Rust Reverse Engineering" (rust) was after "ScrapeGraph AI" (scrap). Moved before. Verified locally: `python3 ./scripts/check-alphabetical.py README.md` → "All sections are alphabetically sorted."
Summary
Restores the ArmorCodex listing in
## Community Plugins → Tools & Integrationsafter it was accidentally dropped during the PR #115 reconcile (commitbc06f70included a single-line-deletion of our entry alongside its+adds). Bundle was added in PR #140 (merged 2026-05-20); only the README line was lost.What's in this PR
README.mdbash+apply_patchper gemini-bot suggestionplugins/armoriq/armorcodex/(lowercased per maintainer convention).codex-plugin/plugin.json,.codex/,.mcp.json,hooks/hooks.json(8 events),scripts/bootstrap.mjs,scripts/hook-router.mjs,scripts/policy-mcp.mjs,scripts/lib/(13 lib modules),README.md,LICENSE(MIT),SECURITY.md,.codexignore,assets/armoriq-logo.pngplugins/armoriq/armorcodex/.agents/plugins/marketplace.jsonarmoriq, pluginarmorcodex, categoryTools & Integrations, source URLhttps://github.com/armoriq/armorCodex.git, follows AgiFlow reference shape)HOL Plugin Scanner — Evidence
Plugin repo: https://github.com/armoriq/armorCodex
Scanner workflow:
.github/workflows/hol-plugin-scanner.ymlusinghashgraph-online/ai-plugin-scanner-action@v1,min_score: 80,fail_on_severity: high, SARIF upload.CI run on plugin main: https://github.com/armoriq/armorCodex/actions/runs/27229800929 (✅ success)
Local scanner run:
Bar: 80+/130 with zero critical/high. Achieved: 92/100, zero critical, zero high.
Note on repo-wide HOL Plugin Scanner — Lint check
This check is failing on this PR but failing identically on every other open PR (#202, #204) since the lint workflow was made mandatory in #199. I ran the lint locally against current
main:plugins/armoriq/armorcodex/: 0The HIGH findings come from
skills/sealos-deploy/scripts/*.mjs(fs.readFileSync/fs.writeFileSyncin install scripts),skills-codex/standards/references/*.md(code-example snippets in docs),archcore.ai/agentopsinstall scripts, and 5 in this repo's ownREADME.md. None are introduced by this PR.Happy to send a follow-up PR with a
--baselinefor the existing 139 findings (standard pattern for this scanner) if useful.Refs
bc06f70🤖 Generated with Claude Code