Skip to content

Restore ArmorCodex entry in Community Plugins (lost during PR #115 reconcile) - #189

Closed
Harihara04sudhan wants to merge 6 commits into
hashgraph-online:mainfrom
Harihara04sudhan:restore-armorcodex
Closed

Restore ArmorCodex entry in Community Plugins (lost during PR #115 reconcile)#189
Harihara04sudhan wants to merge 6 commits into
hashgraph-online:mainfrom
Harihara04sudhan:restore-armorcodex

Conversation

@Harihara04sudhan

@Harihara04sudhan Harihara04sudhan commented Jun 5, 2026

Copy link
Copy Markdown

Summary

Restores the ArmorCodex listing in ## Community Plugins → Tools & Integrations after it was accidentally dropped during the PR #115 reconcile (commit bc06f70 included 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

File / path Change
README.md Restore ArmorCodex entry under Community Plugins → Tools & Integrations (alphabetically between Antigravity and AxonFlow), with backticked bash + apply_patch per gemini-bot suggestion
plugins/armoriq/armorcodex/ (lowercased per maintainer convention) Complete plugin bundle: .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.png
plugins/armoriq/armorcodex/.agents/plugins/marketplace.json Per-plugin marketplace metadata (owner armoriq, plugin armorcodex, category Tools & Integrations, source URL https://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.yml using hashgraph-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:

$ pipx run plugin-scanner scan plugins/armorcodex --format text

── Manifest Validation (28/31) ──
── Security (15/20) ──
── Operational Security (6/6) ──
── Best Practices (9/9) ──
── Marketplace (0/0) ──
── Skill Security (0/0) ──
── Code Quality (10/10) ──

Findings: critical:0, high:0, medium:0, low:1, info:1

━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━
Final Score: 92/100 (A - Excellent)
━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━

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:

  • Total findings: 1333
  • HIGH findings: 139
  • HIGH findings referencing plugins/armoriq/armorcodex/: 0

The HIGH findings come from skills/sealos-deploy/scripts/*.mjs (fs.readFileSync/fs.writeFileSync in install scripts), skills-codex/standards/references/*.md (code-example snippets in docs), archcore.ai/agentops install scripts, and 5 in this repo's own README.md. None are introduced by this PR.

Happy to send a follow-up PR with a --baseline for the existing 139 findings (standard pattern for this scanner) if useful.

Refs

🤖 Generated with Claude Code

@gemini-code-assist gemini-code-assist Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

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.

Comment thread README.md Outdated
- [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.

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

medium

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.

Harihara04sudhan added a commit to Harihara04sudhan/awesome-codex-plugins that referenced this pull request Jun 9, 2026
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
@Harihara04sudhan

Copy link
Copy Markdown
Author

@internet-dot — all asks addressed. Single summary:

What's now in the PR

Ask Status
Complete plugin bundle at plugins/armoriq/armorCodex/ ✅ 25 files: .codex-plugin/plugin.json, .codex/, .mcp.json, hooks/hooks.json (8 events), scripts/ (bootstrap + hook-router + policy-mcp + 13 lib modules), README.md, LICENSE, SECURITY.md, .codexignore, assets/armoriq-logo.png
Per-plugin .agents/plugins/marketplace.json ✅ Added (owner armoriq, plugin armorcodex, category Tools & Integrations, source URL https://github.com/armoriq/armorCodex.git, follows AgiFlow shape)
HOL Plugin Scanner workflow in our plugin repo ✅ Live on armoriq/armorCodex main, CI run 27229800929 green, score 92/100, 0 high/critical
Alphabetical Order Check ✅ Passing on this PR
Gemini bot suggestion (backticks on bash / apply_patch) ✅ Applied

What's excluded from the bundle (intentionally)

  • node_modules/ — resolved at runtime from package-lock.json
  • tests/ — moved out of the plugin distribution dir per scanner compliance; lives at repo root in the source repo

On the repo-wide HOL Plugin Scanner — Lint check

It is failing on this PR but the failure is repo-wide (failing identically on #202, #204, every open PR since the workflow was made mandatory in #199). I ran the lint locally against current main:

  • Total findings: 1333
  • HIGH findings: 139
  • HIGH findings referencing plugins/armoriq/armorCodex/: 0

The HIGH findings come from skills/sealos-deploy/scripts/*.mjs, skills-codex/standards/references/*.md, archcore.ai, agentops install scripts, and 5 in the repo's own README.md. None are introduced by this PR (which adds 1 README line + the standard bundle).

Standard fix for this scenario is --baseline mode: baseline the existing 139 highs against main so only NEW findings from a PR fail. Happy to send a follow-up PR with that baseline if it'd help.

Ready for re-review whenever you have a moment.

@Harihara04sudhan

Copy link
Copy Markdown
Author

@internet-dot — addressed both review asks.

1. Scanner evidence in PR description — added. PR body now includes:

2. Lowercase repo name in path — renamed plugins/armoriq/armorCodex/plugins/armoriq/armorcodex/ to match the dominant convention in this repo (ai-plugin, codex-mem, unity-agent-workflows, call-e-integrations, etc.). 30 files renamed in commit f1befbe9. The upstream GitHub URL in plugin.json / marketplace.json still references the canonical https://github.com/armoriq/armorCodex.git (capital C preserved for the actual repo).

Ready for re-review.

Harihara04sudhan added a commit to Harihara04sudhan/awesome-codex-plugins that referenced this pull request Jun 15, 2026
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
@Harihara04sudhan

Copy link
Copy Markdown
Author

@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.

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.
@Harihara04sudhan

Copy link
Copy Markdown
Author

@internet-dot — added both registry entries in commit 845478e6:

1. plugins.json — new entry under Tools & Integrations:

{
  "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 total: 112 → 113. Inserted alphabetically right before AxonFlow.

2. .agents/plugins/marketplace.json — new entry:

{
  "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."
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.

1 participant