Skip to content

GH#13901: tighten es2016-es2017.md — remove structural noise#18

Merged
johnwaldo merged 1 commit intomainfrom
chore/GH-13901-tighten-es2016-es2017
Mar 30, 2026
Merged

GH#13901: tighten es2016-es2017.md — remove structural noise#18
johnwaldo merged 1 commit intomainfrom
chore/GH-13901-tighten-es2016-es2017

Conversation

@johnwaldo
Copy link
Copy Markdown
Owner

Summary

Remove structural noise from .agents/tools/programming/modern-javascript-skill/es2016-es2017.md (192 → 182 lines):

  • Redundant feature-list summary line (duplicated by section headers below)
  • Trivia sentence "The smallest ECMAScript release with just two features."
  • --- horizontal rule between ES2016/ES2017 (redundant with ## heading)
  • "Syntactic sugar over Promises for cleaner asynchronous code." (prose noise)
  • "Low-level multi-threading primitives (advanced)." (prose noise)

All code blocks, inline comments, and actionable knowledge preserved intact.

Runtime Testing

Risk level: Low (docs-only change, no code execution paths). Self-assessed.

Verification: wc -l before=192, after=182. git diff confirms only prose/structural lines removed — no code block content touched.

Closes marcusquinn#13901


aidevops.sh v3.5.402 plugin for OpenCode v1.3.3 with claude-sonnet-4-6 spent 1m and 2,737 tokens on this as a headless worker.

Remove redundant summary line, trivia sentence, horizontal rule separator,
and non-actionable prose descriptors. All code blocks and actionable
knowledge preserved. 192 → 182 lines.

Closes marcusquinn#13901
@github-actions
Copy link
Copy Markdown

Maintainer Gate: BLOCKED

This PR cannot be merged because:

Issue marcusquinn#13901: API call failed — cannot verify assignee/labels.
What to do:

  1. A maintainer must review the linked issue(s) and remove the needs-maintainer-review label
  2. The issue(s) must have an assignee before work begins
  3. This check re-runs automatically when the issue is updated

This is an automated check. See issue-triage-gate.yml for the triage policy.

@github-actions
Copy link
Copy Markdown

🔍 Code Quality Report

�[0;35m[MONITOR]�[0m Code Review Monitoring Report

SonarCloud: 0 bugs, 0 vulnerabilities, 1 code smells

Mon Mar 30 06:01:31 UTC 2026: Code review monitoring started
Mon Mar 30 06:01:32 UTC 2026: SonarCloud - Bugs: 0, Vulnerabilities: 0, Code Smells: 1

📈 Current Quality Metrics

  • BUGS: 0
  • CODE SMELLS: 1
  • VULNERABILITIES: 0

Generated on: Mon Mar 30 06:01:34 UTC 2026


Generated by AI DevOps Framework Code Review Monitoring

@johnwaldo
Copy link
Copy Markdown
Owner Author

Done

What: Removed structural noise from es2016-es2017.md (192 → 182 lines). Closes marcusquinn#13901.

Testing Evidence: Self-assessed (Low risk — docs-only). All code blocks and actionable knowledge preserved intact.

Key decisions: File classified as reference corpus — content not compressed, only non-actionable structural noise removed (summary line, trivia sentence, redundant --- separator, two prose descriptor sentences).

Files changed: .agents/tools/programming/modern-javascript-skill/es2016-es2017.md


aidevops.sh v3.5.402 plugin for OpenCode v1.3.3 with claude-sonnet-4-6 spent 4m and 6,327 tokens on this as a headless worker.

@johnwaldo johnwaldo deleted the chore/GH-13901-tighten-es2016-es2017 branch March 30, 2026 06:05
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.

simplification: tighten agent doc ES2016 & ES2017 Features (.agents/tools/programming/modern-javascript-skill/es2016-es2017.md, 192 lines)

1 participant