@yegor256/dogent 0.12.0 → 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.0",
+  "version": "0.12.2",
   "dependencies": {
     "minimist": "^1.2.8",
     "pretty-ms": "^7.0.1"

package/src/defaults.js

@@ -40,7 +40,12 @@ class Defaults {
       .split('\n')
       .map((line) => line.trim())
       .filter((line) => line !== '' && !line.startsWith('#'))
-      .flatMap((line) => line.split(/\s+/u));
+      .flatMap((line) => {
+        const separator = line.search(/\s/u);
+        return separator < 0
+          ? [line]
+          : [line.slice(0, separator), line.slice(separator).trimStart()];
+      });
   }
 }
 

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/dead-import.js

@@ -61,7 +61,7 @@ class DeadImport {
     return 'Fix or remove the @path import so it points to a real file, and break any circular or overly deep import chain the host tool cannot resolve.';
   }
   prompt() {
-    return `${this.id}: flag any @path/to/file import that points to no file on disk`;
+    return `${this.id}: flag any @path/to/file import that points to no file on disk; only an @-prefixed token counts as an import, so never treat a bare path in prose as one`;
   }
   violations(document) {
     const uri = document.uri();

package/src/rules/jargon.js

@@ -48,6 +48,8 @@ const defined = (masked) => {
   while (hit !== null) {
     if (hit.groups.acronym) {
       found.add(hit.groups.acronym);
+    } else if (/^[A-Z]{2,}$/u.test(hit.groups.gloss)) {
+      found.add(hit.groups.gloss);
     } else {
       found.add(initials(hit.groups.gloss));
     }
@@ -64,10 +66,11 @@ const undefining = (acronym, scope) => !scope.known.has(acronym) &&
  *
  * Flags an acronym that lands in prose without ever being expanded. An
  * acronym counts as defined when the document, anywhere, follows it with
- * a parenthetical gloss, as in "RBAC (role-based access control)", or when
+ * a parenthetical gloss, as in "RBAC (role-based access control)", when
  * a parenthetical's word initials spell it, as in "AAA pattern
- * (Arrange-Act-Assert)", so a single expansion licenses every later
- * mention. Well-known acronyms sit
+ * (Arrange-Act-Assert)", or when the expansion precedes a parenthetical
+ * acronym, as in "Virtual Private Network (VPN)", so a single expansion
+ * licenses every later mention. Well-known acronyms sit
  * in a built-in allowlist and pass untouched. Only the first unexpanded
  * occurrence of each acronym is reported. Its prompt hands non-acronym
  * domain jargon, the rare nouns a reader cannot parse, to the AI oracle.

package/src/rules/positive.js

@@ -19,7 +19,12 @@ const mask = require('../mask');
  * "Only use real data" beats "Don't use mock data". Its prompt hands
  * subtler bans, those carrying no head keyword, to the AI
  * oracle, which rewrites a prohibition with no keyword as a positive
- * command.
+ * 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() {
@@ -29,7 +34,7 @@ class Positive {
     return 'Rewrite a prohibition as a positive imperative stating what to do, since a ban forces the model to process the forbidden idea first.';
   }
   prompt() {
-    return `${this.id}: flag any instruction phrased as a prohibition, including bans carrying no fixed keyword, and rewrite each as a positive imperative`;
+    return `${this.id}: flag an instruction only when it forbids or negates an action, including a ban that carries no fixed keyword, and rewrite each as a positive imperative; leave an affirmative imperative that already states what to do untouched, returning nothing for it`;
   }
   violations(document) {
     const uri = document.uri();
@@ -41,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.0` marks an unreleased build straight from source.
+ * The default `0.12.2` marks an unreleased build straight from source.
  */
-const version = '0.12.0';
+const version = '0.12.2';
 
 module.exports = version;