Skip to content

Add SeparateWeb Capture to community plugins#133

Closed
AUN-PN wants to merge 4 commits into
hashgraph-online:mainfrom
AUN-PN:main
Closed

Add SeparateWeb Capture to community plugins#133
AUN-PN wants to merge 4 commits into
hashgraph-online:mainfrom
AUN-PN:main

Conversation

@AUN-PN
Copy link
Copy Markdown
Contributor

@AUN-PN AUN-PN commented May 19, 2026

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

  • README.md entry is alphabetically sorted within its category
  • .codex-plugin/plugin.json exists and is valid JSON
  • composerIcon field is set in plugin.json interface section
  • Icon file exists at the path referenced by composerIcon
  • All links in the README entry are valid
  • No placeholder or TODO values in plugin.json

AUN-PN and others added 4 commits May 15, 2026 15:33
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.
@AUN-PN AUN-PN closed this May 19, 2026
Copy link
Copy Markdown
Contributor

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

The formula for simple majority is incorrect for odd numbers. ceil(N / 2) + 1 requires unanimity for $N=1$ and $N=3$ (e.g., for $N=3$, it requires 3 passes instead of 2). The standard formula for a simple majority is floor(N / 2) + 1.

Suggested change
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}
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

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.

Suggested change
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 ))
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

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.

Suggested change
age_days=$(( (now_epoch - mtime_epoch) / 86400 ))
age_days=$(( (now_epoch - ${mtime_epoch:-0}) / 86400 ))

Copy link
Copy Markdown

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Choose a reason for hiding this comment

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

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

Comment thread README.md
- [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.
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

P2 Badge 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 👍 / 👎.

@kilo-code-bot
Copy link
Copy Markdown

kilo-code-bot Bot commented May 19, 2026

Code Review Summary

Status: 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)
  • plugins/Kanevry/session-orchestrator/skills/persona-panel/SKILL.md (new)
  • plugins/Kanevry/session-orchestrator/skills/persona-panel/persona-format.md (new)
  • plugins/Kanevry/session-orchestrator/skills/wave-executor/wave-loop.md
  • plugins/boshu2/agentops/skills-codex/ship-loop/SKILL.md (new)
  • plugins/boshu2/agentops/skills-codex/ship-loop/prompt.md (new)
  • plugins/boshu2/agentops/skills-codex/evolve/SKILL.md
  • Various manifest/hash updates (.agentops-manifest.json, plugin configs)

Reviewed by laguna-m.1-20260312:free · 845,672 tokens

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