Skip to content

improve monitoring-security skill with concrete configs and validation#1

Open
fernandezbaptiste wants to merge 1 commit into
rayjun:mainfrom
fernandezbaptiste:improve-monitoring-security-skill
Open

improve monitoring-security skill with concrete configs and validation#1
fernandezbaptiste wants to merge 1 commit into
rayjun:mainfrom
fernandezbaptiste:improve-monitoring-security-skill

Conversation

@fernandezbaptiste
Copy link
Copy Markdown

@fernandezbaptiste fernandezbaptiste commented Apr 9, 2026

hey @rayjun, nice work building out a full set of Claude Code skills for your ai-coding-context project. Kudos on the structured approach with 6 skills covering everything from debugging to vault management! I've just starred it.

ran your monitoring-security skill through agent evals and spotted a few quick wins that took it from ~65% to ~86% performance:

  • added copy-paste-ready configuration examples for Docker Compose (network isolation + non-root), Prometheus Basic Auth, Grafana RBAC, and Nginx TLS reverse proxy, replacing the abstract guidance that scored low on actionability

  • replaced the flat 4-step implementation list with a sequenced 5-step protocol (network isolation, auth, TLS, secrets, verification) with explicit validation checkpoints and expected outputs at each stage

  • condensed the redundant core patterns section (which duplicated the quick reference table) and added a concrete verification step with 6 specific commands to confirm TLS, auth, security headers, and container user

this means you and your contributors get an instant quality signal and improvement suggestions before you review.

these were easy changes to bring the skill in line with what performs well against Anthropic's best practices. honest disclosure, I work at tessl.io where we build tooling around this. not a pitch, just fixes that were straightforward to make.

if you want to review your other skills, two options: I can open a follow-up PR with a GitHub Action that auto-scores skill.md changes on every PR (no signup, no token needed - it runs fully in CI with pinned versions you can inspect). this means you and your contributors get an instant quality signal and improvement suggestions before you review.

Alternatively, if you'd rather do it yourself, spin up Claude Code and run tessl skill review --optimize for your other skills.

happy to answer any questions on the changes.

@rayjun
Copy link
Copy Markdown
Owner

rayjun commented Apr 14, 2026

Are you using AI to create PRs in bulk?

rayjun added a commit that referenced this pull request May 24, 2026
Four small follow-ups to Round 6 P0:

- AGENTS.md §6: merge process rules #1 and #6 (one was a restatement of
  the other) — six rules become five
- AGENTS.md §9: drop the standalone "其他约定" section (only three lines
  about output style) and fold it into §4 as a "### 输出风格" subsection;
  AGENTS.md now runs §0–§8 contiguously
- skills/obsidian-writer: prepend a "示例 skill" callout above the body,
  making clear this skill is here as a per-vault AGENTS.md design
  showcase rather than a feature this repo uses
- README.md: skill count drops from 6 to 5 (Round 5 removed
  monitoring-security but README still claimed 6); also notes
  obsidian-writer's demo status

Measured: AGENTS.md 5246 → 5092 chars (~50 tokens), orient-session
output 2330 → 2226 bytes (~35 tokens). Round 6 (P0 + P1) cumulative
savings ~205 tokens per session.

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
rayjun added a commit that referenced this pull request May 25, 2026
Four edits removing the same rule restated in multiple places inside
AGENTS.md:

- §6 superpowers note: collapse the eight-skill name list down to "★
  marks superpowers-provided" and put a ★ suffix on those rows in the
  step table — readers can now tell external from in-repo skills at a
  glance instead of cross-referencing the prelude
- §6 prelude: drop the standalone "跳步前必须明确询问用户确认,
  禁止静默跳过" line; the same rule is process rule #1 below
- §2 matrix footnote: change "必做 = 禁止静默跳过, 跳须询问用户" to
  "必做 = 必须执行 (跳步规则见 §6)" so §2 and §6 stop saying the same
  thing in slightly different words
- §8: remove the "Harness 提醒" bullet list (status-reminder.sh /
  drift-detector.sh / session-end.sh) — each hook's own header comment
  already says exactly the same thing, and the canonical registration
  list lives in .claude/settings.json

Measured: AGENTS.md drops 5092 → 4708 chars (~128 tokens). "禁止静默"
mentions inside AGENTS.md drop from 3 to 1 (only process rule #1
remains). danger-patterns 25/25 still PASS.

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
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