@review-vir/common 3.0.0 → 3.1.1

package/dist/code-owners-markdown.d.ts

@@ -0,0 +1,16 @@
+/**
+ * Defines the Markdown that identifies a code owners block in a pull request description.
+ *
+ * @category Main
+ */
+export declare const codeOwnersMarkdown: Readonly<{
+    blockEnd: "<!-- code owners end -->";
+    blockStart: "<!-- code owners start -->";
+    heading: "## Code Owners";
+}>;
+/**
+ * Formats the code owners content that pull-request-vir writes into a pull request description.
+ *
+ * @category Main
+ */
+export declare function createCodeOwnersMarkdownBlock(content: string): string;

package/dist/code-owners-markdown.js

@@ -0,0 +1,23 @@
+/**
+ * Defines the Markdown that identifies a code owners block in a pull request description.
+ *
+ * @category Main
+ */
+export const codeOwnersMarkdown = Object.freeze({
+    blockEnd: '<!-- code owners end -->',
+    blockStart: '<!-- code owners start -->',
+    heading: '## Code Owners',
+});
+/**
+ * Formats the code owners content that pull-request-vir writes into a pull request description.
+ *
+ * @category Main
+ */
+export function createCodeOwnersMarkdownBlock(content) {
+    return [
+        codeOwnersMarkdown.blockStart,
+        codeOwnersMarkdown.heading,
+        content,
+        codeOwnersMarkdown.blockEnd,
+    ].join('\n');
+}

package/dist/index.d.ts

@@ -1 +1,3 @@
+export * from './code-owners-markdown.js';
 export * from './parse-description-users.js';
+export * from './parse-inserted-code-owners.js';

package/dist/index.js

@@ -1 +1,3 @@
+export * from './code-owners-markdown.js';
 export * from './parse-description-users.js';
+export * from './parse-inserted-code-owners.js';

package/dist/parse-description-users.js

@@ -16,5 +16,8 @@ export function parseDescriptionUsers({ triggerText, bodyText, }) {
     }
     const [, restrictedMatch,] = safeMatch(match, /((?:@[\w-]+[^\w@]*)+)/);
     const userTags = Array.from(restrictedMatch?.matchAll(/@[\w-]+/g) || []);
-    return userTags.map((userTag) => removePrefix({ value: userTag[0], prefix: '@' }));
+    return userTags.map((userTag) => removePrefix({
+        value: userTag[0],
+        prefix: '@',
+    }));
 }

package/dist/parse-inserted-code-owners.d.ts

@@ -0,0 +1,27 @@
+/**
+ * Parses the code owners that pull-request-vir inserts into a pull request's _raw markdown_ body.
+ *
+ * Pull-request-vir wraps its code owners in HTML comment delimiters and lists each owner on its own
+ * line followed by a collapsible list of their owned files:
+ *
+ *     <!-- code owners start -->
+ *     ## Code Owners
+ *     (at)username1
+ *     <details>
+ *     <summary>Owned files</summary>
+ *
+ *     - [some/file.ts](https://...)
+ *
+ *     </details>
+ *     (at)username2
+ *     ...
+ *     <!-- code owners end -->
+ *
+ * Returns the owner usernames (e.g. `['username1', 'username2']`), or `undefined` when the body
+ * contains no inserted code owners block (so callers can fall back to other detection).
+ *
+ * This must run against the raw markdown `body`, not the rendered `bodyText`: GitHub strips HTML
+ * comments (and the `<details>` structure) out of `bodyText`, removing the delimiters that bound
+ * the block and interleaving the owners with their owned-file text.
+ */
+export declare function parseInsertedCodeOwners(body: string): string[] | undefined;

package/dist/parse-inserted-code-owners.js

@@ -0,0 +1,47 @@
+import { removeDuplicates, removePrefix, safeMatch } from '@augment-vir/common';
+import { codeOwnersMarkdown } from './code-owners-markdown.js';
+const insertedCodeOwnersRegExp = new RegExp(String.raw `${codeOwnersMarkdown.blockStart}([\S\s]*?)${codeOwnersMarkdown.blockEnd}`, 'i');
+const fileLinkLineRegExp = /^\s*-\s/;
+const userTagRegExp = /@[\w-]+/g;
+/**
+ * Parses the code owners that pull-request-vir inserts into a pull request's _raw markdown_ body.
+ *
+ * Pull-request-vir wraps its code owners in HTML comment delimiters and lists each owner on its own
+ * line followed by a collapsible list of their owned files:
+ *
+ *     <!-- code owners start -->
+ *     ## Code Owners
+ *     (at)username1
+ *     <details>
+ *     <summary>Owned files</summary>
+ *
+ *     - [some/file.ts](https://...)
+ *
+ *     </details>
+ *     (at)username2
+ *     ...
+ *     <!-- code owners end -->
+ *
+ * Returns the owner usernames (e.g. `['username1', 'username2']`), or `undefined` when the body
+ * contains no inserted code owners block (so callers can fall back to other detection).
+ *
+ * This must run against the raw markdown `body`, not the rendered `bodyText`: GitHub strips HTML
+ * comments (and the `<details>` structure) out of `bodyText`, removing the delimiters that bound
+ * the block and interleaving the owners with their owned-file text.
+ */
+export function parseInsertedCodeOwners(body) {
+    const [, block,] = safeMatch(body, insertedCodeOwnersRegExp);
+    if (block == undefined) {
+        return undefined;
+    }
+    /** Drop owned-file link lines so scoped paths (e.g. `@types/...`) aren't read as owners. */
+    const ownerLines = block
+        .split('\n')
+        .filter((line) => !fileLinkLineRegExp.test(line))
+        .join('\n');
+    const userTags = Array.from(ownerLines.matchAll(userTagRegExp));
+    return removeDuplicates(userTags.map((userTag) => removePrefix({
+        value: userTag[0],
+        prefix: '@',
+    })));
+}

package/package.json

@@ -1,6 +1,6 @@
 {
     "name": "@review-vir/common",
-    "version": "3.0.0",
+    "version": "3.1.1",
     "description": "Common functionality that review-vir primary uses but other external packages need as well.",
     "keywords": [
         "review-vir",
@@ -16,7 +16,7 @@
         "type": "git",
         "url": "git+https://github.com/review-vir/review-vir.git"
     },
-    "license": "(MIT or CC0 1.0)",
+    "license": "(MIT OR CC0-1.0)",
     "author": {
         "name": "electrovir",
         "url": "https://github.com/electrovir"
@@ -35,20 +35,20 @@
         "test:docs": "virmator docs check"
     },
     "dependencies": {
-        "@augment-vir/assert": "^31.1.1",
-        "@augment-vir/common": "^31.1.1"
+        "@augment-vir/assert": "^32.2.1",
+        "@augment-vir/common": "^32.2.1"
     },
     "devDependencies": {
-        "@augment-vir/test": "^31.1.1",
-        "@web/dev-server-esbuild": "^1.0.3",
-        "@web/test-runner": "^0.19.0",
-        "@web/test-runner-commands": "^0.9.0",
-        "@web/test-runner-playwright": "^0.11.0",
-        "@web/test-runner-visual-regression": "^0.10.0",
+        "@augment-vir/test": "^32.2.1",
+        "@web/dev-server-esbuild": "^2.0.0",
+        "@web/test-runner": "^1.0.0",
+        "@web/test-runner-commands": "^1.0.1",
+        "@web/test-runner-playwright": "^1.0.0",
+        "@web/test-runner-visual-regression": "^1.0.1",
         "istanbul-smart-text-reporter": "^1.1.5",
-        "markdown-code-example-inserter": "^3.0.3",
-        "typedoc": "^0.27.4",
-        "typescript": "~5.6.3"
+        "markdown-code-example-inserter": "^3.0.6",
+        "typedoc": "^0.28.20",
+        "typescript": "^6.0.3"
     },
     "engines": {
         "node": ">=22"