Skip to content

docs: add HOL registry links and broker plugin example#33

Merged
internet-dot merged 1 commit intomainfrom
feat/hol-registry-linking
Apr 1, 2026
Merged

docs: add HOL registry links and broker plugin example#33
internet-dot merged 1 commit intomainfrom
feat/hol-registry-linking

Conversation

@internet-dot
Copy link
Copy Markdown
Collaborator

Adds HOL Plugin Registry links throughout README and includes the Registry Broker Codex Plugin as a scanned example showing 130/130 (A+ Perfect) trust score.

- Link to HOL Plugin Registry in intro and resources
- Add Registry Broker Codex Plugin as scanned example (100/100)
- Update scanner version in example output to 1.4.0
- Add HOL Standards docs link to resources
Copy link
Copy Markdown

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

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 updates the README.md to reflect version 1.4.0, adds a new example section for the HOL Registry Broker Plugin, and updates various links to the HOL ecosystem. A review comment points out that the example scan score should be updated to 100/100 to match the scanner's normalization logic.

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

━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━
Final Score: 130/130 (A+ - Perfect)
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

medium

The example output shows a score of 130/130, but the scanner's implementation in src/codex_plugin_scanner/scanner.py (line 90) normalizes the final score to a percentage (0-100). To accurately reflect the tool's output, this should be updated to 100/100. Additionally, ensure the grade and label (A+ - Perfect) match the output generated by the tool for a 100% score, as the existing example on line 190 uses A - Excellent for the same percentage.

Suggested change
Final Score: 130/130 (A+ - Perfect)
Final Score: 100/100 (A+ - Perfect)

@internet-dot internet-dot merged commit 8b81caa into main Apr 1, 2026
19 of 20 checks passed
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.

2 participants