@yegor256/dogent 0.12.1 → 0.12.2

package/README.md

@@ -53,7 +53,7 @@ Most rewrite prompts for you or score a file, while we enforce
 Run it on any manifesto file, no installation required:
 
 ```bash
-npx @yegor256/dogent@0.11.0 SKILL.md
+npx @yegor256/dogent@0.12.1 SKILL.md
 ```
 
 Point it at a directory to lint the default manifestos it holds
@@ -266,7 +266,7 @@ Reference `dogent` as a remote hook in `.pre-commit-config.yaml`:
 ```yaml
 repos:
   - repo: https://github.com/yegor256/dogent
-    rev: 0.11.0
+    rev: 0.12.1
     hooks:
       - id: dogent
 ```

package/package.json

@@ -40,7 +40,7 @@
     "lint": "eslint .",
     "test": "mocha 'test/**/*.js' --timeout 60000"
   },
-  "version": "0.12.1",
+  "version": "0.12.2",
   "dependencies": {
     "minimist": "^1.2.8",
     "pretty-ms": "^7.0.1"

package/src/oracle.js

@@ -14,7 +14,9 @@ const Answer = require('./answer');
  * The AI second opinion. Wraps the rules and a chat endpoint, builds one
  * prompt from a document, asks the endpoint, and parses the reply into
  * violations paired with the token usage the model reported. Mirrors a
- * rule, but consults a model instead of guessing.
+ * rule, but consults a model instead of guessing. Lets each rule veto an
+ * oracle flag it knows to be false, so a deterministic guard overrides
+ * the model.
  */
 class Oracle {
   constructor(rules, chat) {
@@ -24,7 +26,11 @@ class Oracle {
   async violations(document) {
     const reply = await this.chat.answer(new Prompt(this.rules, document).text());
     return {
-      found: new Answer(reply.content).violations(),
+      found: new Answer(reply.content).violations().filter(
+        (violation) => !this.rules.some(
+          (rule) => rule.suppress?.(violation, document)
+        )
+      ),
       usage: reply.usage
     };
   }

package/src/rules/command.js

@@ -15,6 +15,8 @@ const Region = require('../region');
  * checker can only guess: it flags lines that open with a pronoun or
  * end with a question mark, both signs of description, not order. Its
  * prompt hands the subtler imperative-mood judgement to the AI oracle.
+ * A deterministic guard then drops any oracle flag on a line that
+ * shows neither sign, so a base-form imperative is never flagged.
  */
 class Command {
   constructor() {
@@ -37,16 +39,7 @@ class Command {
     });
   }
   judge(text, line, uri) {
-    const clean = text.replace(/^\s*(?:[-*+]|\d+\.)\s+/u, '').trim();
-    if (clean === '') {
-      return [];
-    }
-    const first = clean
-      .split(/\s+/u)[0]
-      .toLowerCase()
-      .replace(/[^a-z]/gu, '');
-    const weak = /^(?:i|you|we|they|he|she|it|this|that|these|those|there|here)$/u;
-    if (!weak.test(first) && clean.slice(-1) !== '?') {
+    if (!this.describes(text)) {
       return [];
     }
     return [new Violation(
@@ -56,6 +49,25 @@ class Command {
       new Region(uri, line, 1)
     )];
   }
+  suppress(violation, document) {
+    if (violation.rule !== this.id) {
+      return false;
+    }
+    const lines = document.text().split('\n');
+    return !this.describes(lines[violation.spot.line() - 1] || '');
+  }
+  describes(text) {
+    const clean = text.replace(/^\s*(?:[-*+]|\d+\.)\s+/u, '').trim();
+    if (clean === '') {
+      return false;
+    }
+    const first = clean
+      .split(/\s+/u)[0]
+      .toLowerCase()
+      .replace(/[^a-z]/gu, '');
+    const weak = /^(?:i|you|we|they|he|she|it|this|that|these|those|there|here)$/u;
+    return weak.test(first) || clean.slice(-1) === '?';
+  }
 }
 
 module.exports = Command;

package/src/rules/positive.js

@@ -21,6 +21,10 @@ const mask = require('../mask');
  * oracle, which rewrites a prohibition with no keyword as a positive
  * command. The prompt demands an actual negation before flagging, so
  * an affirmative imperative that already states what to do stays clean.
+ * Because the model still misreads plain imperatives as bans, a
+ * deterministic guard then drops any oracle flag on a line that carries
+ * no negation token at all, so an affirmative imperative can never be
+ * reported regardless of what the model returns.
  */
 class Positive {
   constructor() {
@@ -42,6 +46,17 @@ class Positive {
       frontmatter: () => []
     });
   }
+  suppress(violation, document) {
+    if (violation.rule !== this.id) {
+      return false;
+    }
+    const lines = document.text().split('\n');
+    return !this.negated(lines[violation.spot.line() - 1] || '');
+  }
+  negated(text) {
+    const regex = /\b(?:do not|don't|never|avoid|refrain from|must not|no longer|no|not)\b/iu;
+    return regex.test(mask(text));
+  }
   scan(text, line, uri) {
     const regex = /^(?<marker>\s*(?:[-*+]|\d+\.)\s+)?(?:do not|don't|never|avoid|refrain from|must not|no longer)\b/iu;
     const hit = regex.exec(mask(text));

package/src/version.js

@@ -9,8 +9,8 @@
  * Version.
  *
  * The current release of dogent, replaced on every release by rultor.
- * The default `0.12.1` marks an unreleased build straight from source.
+ * The default `0.12.2` marks an unreleased build straight from source.
  */
-const version = '0.12.1';
+const version = '0.12.2';
 
 module.exports = version;