Skip to content

tooling: turn line-capture and line-analyze into skills - #288

Open
clins1994 wants to merge 1 commit into
beeper:mainfrom
clins1994:tooling/skills-capture-analyze
Open

clins1994 wants to merge 1 commit into
beeper:mainfrom
clins1994:tooling/skills-capture-analyze

Conversation

@clins1994

Copy link
Copy Markdown
Contributor

Moves line-capture and line-analyze from commands to skills in this repo, with their scripts next to them, as asked on #280. Please review this one first: it is the base of a stack of five PRs that bring the LINE skills here.


🇬🇧 More details

Summary

  • line-capture and line-analyze are now skills in .agents/skills/, each with its scripts in a scripts/ folder.
  • The four scripts move out of scripts/ and get the updates that scripts: make the LINE extension capture work on macOS, Linux and Windows #278 had: a dependency preflight, and a launcher that works on macOS, Linux and Windows (Git Bash).
  • .claude/skills links to .agents/skills, so Claude Code finds the skills. Codex already reads .agents/skills.

Why

On #280, @highesttt asked to improve the skills here and review them, instead of pointing to a repo outside the project. This stack does that. Each PR adds or updates one part:

  1. This PR: line-capture and line-analyze.
  2. line-test-plan.
  3. line-evidence.
  4. line-beeper-api.
  5. line-implement, and the old .agents/commands folder goes away.

GitHub cannot stack PRs from a fork, so every PR targets main. Until this one merges, the diff of each later PR also shows this one.

Changes

  • .agents/commands/line-capture.md and .agents/commands/line-analyze.md become .agents/skills/line-capture/SKILL.md and .agents/skills/line-analyze/SKILL.md.
  • scripts/line-chrome-capture.sh and scripts/line-cdp-capture.py move to .agents/skills/line-capture/scripts/.
    • New line-capture-preflight.sh checks Chrome (Chromium on Linux), Python 3.8+ and websocket-client. It prints OK or MISSING per item. The launcher refuses to start while something is missing, because without it only the net-log is saved.
    • The launcher finds Chrome and Python on each OS and refuses Windows Chrome from WSL.
    • The CDP recorder releases every page-like target. Before, windows that the extension opens (Settings, group popups) sometimes started frozen with "Debugger paused in another tab".
  • scripts/parse-line-traffic.py and scripts/redact-line-capture.py move to .agents/skills/line-analyze/scripts/. The analyze skill now prefers the CDP capture over the net-log, and redacts before anything is read or quoted.
  • .agents/skills/line-implement/REFERENCE.md points at the new script paths. The full line-implement update comes in PR 5.

How it was tested

Not tested (and why)

  • The launcher was not run from its new path for a full capture. In this repo it finds the repository root from its own location (four folders up), instead of asking git.
  • Linux and Windows were not run for this PR.

Related

🤖 Generated with Claude Code using clins:create-pr

Move the line-capture and line-analyze commands to .agents/skills, with
their scripts next to them, and link .claude/skills to .agents/skills so
Claude Code finds them.

The capture launcher now runs a dependency preflight first and works on
macOS, Linux and Windows (Git Bash). The CDP recorder releases every
page-like target so extension windows no longer open frozen.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

1 participant