Add SeparateWeb Capture to community plugins#133
Conversation
Add Unity Agent Workflows to the Development & Workflow community plugin list following CONTRIBUTING.md.
Adds SeparateWeb Capture to the Tools & Integrations section. SeparateWeb gives Codex real webpage context through full-page screenshots, UI crops, and JSON manifests for frontend review and visual QA.
There was a problem hiding this comment.
Code Review
This pull request introduces significant updates across multiple plugins, most notably the addition of the persona-panel skill for multi-persona content review and the ship-loop skill for streamlined internal PR cycles. It also integrates the 'Intent-to-Loop Hexagon' architectural pattern into several core skills like planning, discovery, and validation. Feedback from the review correctly identified a logic error in the majority threshold calculation for the persona panel, a missing crypto reference for UUID generation, and a potential bash syntax error in the evolve skill's stale-kill check.
|
|
||
| ### `voting-quorum` (default) | ||
|
|
||
| Deterministic M-of-N threshold. Default M = ceil(N / 2) + 1 (simple majority). Override with |
There was a problem hiding this comment.
The formula for simple majority is incorrect for odd numbers. ceil(N / 2) + 1 requires unanimity for floor(N / 2) + 1.
| Deterministic M-of-N threshold. Default M = ceil(N / 2) + 1 (simple majority). Override with | |
| Deterministic M-of-N threshold. Default M = floor(N / 2) + 1 (simple majority). Override with |
|
|
||
| **Run ID generation (H1 security guard):** | ||
| ```js | ||
| const runId = randomUUID().slice(0, 8); // format: [a-z0-9-]{8} |
There was a problem hiding this comment.
randomUUID() is not a global function in Node.js or standard browser environments. It should be accessed via the crypto object to ensure the specification is technically accurate for implementation.
| const runId = randomUUID().slice(0, 8); // format: [a-z0-9-]{8} | |
| const runId = crypto.randomUUID().slice(0, 8); // format: [a-z0-9-]{8} |
| local mtime_epoch now_epoch age_days | ||
| mtime_epoch=$(stat -c %Y "$path" 2>/dev/null || stat -f %m "$path" 2>/dev/null) | ||
| now_epoch=$(date +%s) | ||
| age_days=$(( (now_epoch - mtime_epoch) / 86400 )) |
There was a problem hiding this comment.
If the stat command fails to retrieve the modification time (e.g., due to platform differences or permission issues), mtime_epoch will be empty, causing a syntax error in the bash arithmetic expression. Providing a default value prevents the script from crashing.
| age_days=$(( (now_epoch - mtime_epoch) / 86400 )) | |
| age_days=$(( (now_epoch - ${mtime_epoch:-0}) / 86400 )) |
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 563dc81034
ℹ️ About Codex in GitHub
Codex has been enabled to automatically 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 👍.
When you sign up for Codex through ChatGPT, Codex can also answer questions or update the PR, like "@codex address that feedback".
| - [Remotion Plugin](https://github.com/tim-osterhus/codex-remotion-plugin) - Build parameterized Remotion videos in Codex with the official Remotion docs MCP, composition scaffolding, and a data-driven launch-video workflow. | ||
| - [ru-text](https://github.com/talkstream/ru-text) - Russian text quality — ~1,040 rules for typography, info-style, editorial, UX writing, and business correspondence. | ||
| - [Rust Reverse Engineering](https://github.com/jingjing2222/rust-reverse-engineering-skill) - Reverse engineer Rust binaries and libraries: triage targets, demangle symbols, recover crate namespaces, and map panic, unwind, async, and FFI paths. | ||
| - [SeparateWeb Capture](https://github.com/AUN-PN/SeparateWeb) - Give Codex eyes on real webpages with full-page screenshots, UI crops, and JSON manifests for frontend visual QA. |
There was a problem hiding this comment.
Regenerate registry artifacts for SeparateWeb
Adding this README entry makes scripts.generate_plugins_json.parse_plugins() return 81 plugins, but the committed plugins.json and .agents/plugins/marketplace.json still contain 80 entries and no AUN-PN/SeparateWeb record, and there is no mirrored plugins/AUN-PN/SeparateWeb bundle. In contexts that install from the generated registry/marketplace rather than the README, SeparateWeb will not be discoverable or installable despite being advertised here; please rerun the generator and include the synced artifacts.
Useful? React with 👍 / 👎.
Code Review SummaryStatus: No Issues Found | Recommendation: Merge The PR introduces the persona-panel skill for multi-agent content review with comprehensive security guards (M1-M3, H1-H4), the ship-loop skill for internal PR fast-lane cycles, and updates to evolve for stale-kill auto-expiration. All changes are documentation and configuration files with no code issues identified. Files Reviewed (7 files)
Reviewed by laguna-m.1-20260312:free · 845,672 tokens |
Summary
Adds SeparateWeb Capture to the Community Plugins list.
SeparateWeb Capture gives Codex eyes on real webpages with full-page screenshots, UI crops, and JSON manifests for frontend review and visual QA.
Checklist
.codex-plugin/plugin.jsonexists and is valid JSONcomposerIconfield is set inplugin.jsoninterface sectioncomposerIconplugin.json