@yegor256/dogent 0.6.0 → 0.6.1

package/README.md

@@ -29,7 +29,7 @@ In short: `agnix` lints the harness, `dogent` lints the prompt.
 Run it on any manifesto file, no installation required:
 
 ```bash
-npx @yegor256/dogent@0.5.1 CLAUDE.md
+npx @yegor256/dogent@0.6.0 CLAUDE.md
 ```
 
 Lint several files at once:
@@ -150,7 +150,7 @@ Reference `dogent` as a remote hook in `.pre-commit-config.yaml`:
 ```yaml
 repos:
   - repo: https://github.com/yegor256/dogent
-    rev: 0.5.1
+    rev: 0.6.0
     hooks:
       - id: dogent
 ```

package/package.json

@@ -40,5 +40,5 @@
     "lint": "eslint .",
     "test": "mocha 'test/**/*.js' --timeout 60000"
   },
-  "version": "0.6.0"
+  "version": "0.6.1"
 }

package/src/rules/empty.js

@@ -15,13 +15,16 @@ const Region = require('../region');
  * A heading is empty when it is immediately followed by another
  * heading or by end-of-file — no prose, bullets, or snippet sits
  * between them.
+ *
+ * The check is standalone and deterministic, so prompt() returns an
+ * empty string and the AI oracle never re-checks this rule.
  */
 class Empty {
   constructor() {
     this.id = 'empty';
   }
   prompt() {
-    return `${this.id}: flag any section heading that carries no instructions beneath it`;
+    return '';
   }
   violations(document) {
     const uri = document.uri();

package/src/rules/index.js

@@ -15,6 +15,7 @@ const Command = require('./command');
 const Punctuation = require('./punctuation');
 const Frontmatter = require('./frontmatter');
 const DeadImport = require('./dead-import');
+const Redundant = require('./redundant');
 
 module.exports = () => [
   new Grouped(),
@@ -26,6 +27,7 @@ module.exports = () => [
   new Command(),
   new Punctuation(),
   new DeadImport(),
+  new Redundant(),
   new Frontmatter(
     'SKILL.md',
     ['name', 'description'],

package/src/rules/redundant.js

@@ -0,0 +1,92 @@
+/*
+ * SPDX-FileCopyrightText: Copyright (c) 2026 Yegor Bugayenko
+ * SPDX-License-Identifier: MIT
+ */
+
+'use strict';
+
+const Violation = require('../violation');
+const Region = require('../region');
+
+const PHRASES = [
+  'be helpful',
+  'be helpful and accurate',
+  'be accurate',
+  'be concise',
+  'be clear',
+  'be polite',
+  'be professional',
+  'write clean code',
+  'write good code',
+  'write readable code',
+  'write maintainable code',
+  'follow best practices',
+  'follow industry best practices',
+  'use best practices',
+  'apply best practices',
+  'use meaningful variable names',
+  'use descriptive variable names',
+  'use good variable names',
+  'handle errors properly',
+  'handle errors gracefully',
+  'handle exceptions properly',
+  'avoid bugs',
+  'avoid mistakes',
+  'think step by step',
+  'do your best',
+  'try your best',
+  'pay attention to detail'
+];
+
+/**
+ * Redundant.
+ *
+ * Flags a line that restates default model behavior, like
+ * "Be helpful and accurate" or "Write clean code". Such filler
+ * burns the context budget and drowns the project-specific
+ * guidance the manifesto exists to carry.
+ *
+ * @todo #15:60min Promote the standalone heuristic into a
+ *  proper AI-oracle check when `OPENAI_API_KEY` is present, so
+ *  redundancy detection covers paraphrases beyond the curated
+ *  blacklist below. The hybrid pattern from `src/rules/command.js`
+ *  is the model: keep the deterministic check as the default,
+ *  let the oracle catch the rest.
+ */
+class Redundant {
+  constructor(phrases = PHRASES) {
+    this.id = 'redundant';
+    this.phrases = phrases;
+  }
+  prompt() {
+    return `${this.id}: flag any line that restates default agent behavior already known to the model, not a project-specific instruction`;
+  }
+  violations(document) {
+    const uri = document.uri();
+    return document.walk({
+      header: () => [],
+      prose: (text, line) => this.judge(text, line, uri),
+      snippet: () => [],
+      bullets: () => [],
+      frontmatter: () => []
+    });
+  }
+  judge(text, line, uri) {
+    const clean = text
+      .replace(/^\s*(?:[-*+]|\d+\.)\s+/u, '')
+      .replace(/[.!?]+\s*$/u, '')
+      .trim()
+      .toLowerCase();
+    if (!this.phrases.includes(clean)) {
+      return [];
+    }
+    return [new Violation(
+      this.id,
+      'error',
+      'generic instruction, model already knows this',
+      new Region(uri, line, 1)
+    )];
+  }
+}
+
+module.exports = Redundant;