lula2 0.7.0 → 0.7.1-nightly.0

package/dist/cli/commands/ui.js

@@ -4217,7 +4217,7 @@ import { join as join7 } from "path";
 import cors from "cors";
 import express2 from "express";
 
-// node_modules/.pnpm/express-rate-limit@8.1.0_express@5.1.0/node_modules/express-rate-limit/dist/index.mjs
+// node_modules/.pnpm/express-rate-limit@8.2.1_express@5.1.0/node_modules/express-rate-limit/dist/index.mjs
 var import_ip_address = __toESM(require_ip_address(), 1);
 import { isIPv6 } from "node:net";
 import { isIPv6 as isIPv62 } from "node:net";
@@ -4687,12 +4687,54 @@ var validations = {
       );
     }
   },
+  knownOptions(passedOptions) {
+    if (!passedOptions) return;
+    const optionsMap = {
+      windowMs: true,
+      limit: true,
+      message: true,
+      statusCode: true,
+      legacyHeaders: true,
+      standardHeaders: true,
+      identifier: true,
+      requestPropertyName: true,
+      skipFailedRequests: true,
+      skipSuccessfulRequests: true,
+      keyGenerator: true,
+      ipv6Subnet: true,
+      handler: true,
+      skip: true,
+      requestWasSuccessful: true,
+      store: true,
+      validate: true,
+      headers: true,
+      max: true,
+      passOnStoreError: true
+    };
+    const validOptions = Object.keys(optionsMap).concat(
+      "draft_polli_ratelimit_headers",
+      // not a valid option anymore, but we have a more specific check for this one, so don't warn for it here
+      // from express-slow-down - https://github.com/express-rate-limit/express-slow-down/blob/main/source/types.ts#L65
+      "delayAfter",
+      "delayMs",
+      "maxDelayMs"
+    );
+    for (const key of Object.keys(passedOptions)) {
+      if (!validOptions.includes(key)) {
+        throw new ValidationError(
+          "ERR_ERL_UNKNOWN_OPTION",
+          `Unexpected configuration option: ${key}`
+          // todo: suggest a valid option with a short levenstein distance?
+        );
+      }
+    }
+  },
   /**
    * Checks the options.validate setting to ensure that only recognized
    * validations are enabled or disabled.
    *
    * If any unrecognized values are found, an error is logged that
-   * includes the list of supported vaidations.
+   * includes the list of supported validations.
    */
   validationsConfig() {
     const supportedValidations = Object.keys(this).filter(
@@ -4860,6 +4902,7 @@ var parseOptions = (passedOptions) => {
   const notUndefinedOptions = omitUndefinedProperties(passedOptions);
   const validations2 = getValidations(notUndefinedOptions?.validate ?? true);
   validations2.validationsConfig();
+  validations2.knownOptions(passedOptions);
   validations2.draftPolliHeaders(
     // @ts-expect-error see the note above.
     notUndefinedOptions.draft_polli_ratelimit_headers

package/dist/cli/server/index.js

@@ -4194,7 +4194,7 @@ var init_spreadsheetRoutes = __esm({
 import cors from "cors";
 import express2 from "express";
 
-// node_modules/.pnpm/express-rate-limit@8.1.0_express@5.1.0/node_modules/express-rate-limit/dist/index.mjs
+// node_modules/.pnpm/express-rate-limit@8.2.1_express@5.1.0/node_modules/express-rate-limit/dist/index.mjs
 var import_ip_address = __toESM(require_ip_address(), 1);
 import { isIPv6 } from "node:net";
 import { isIPv6 as isIPv62 } from "node:net";
@@ -4664,12 +4664,54 @@ var validations = {
       );
     }
   },
+  knownOptions(passedOptions) {
+    if (!passedOptions) return;
+    const optionsMap = {
+      windowMs: true,
+      limit: true,
+      message: true,
+      statusCode: true,
+      legacyHeaders: true,
+      standardHeaders: true,
+      identifier: true,
+      requestPropertyName: true,
+      skipFailedRequests: true,
+      skipSuccessfulRequests: true,
+      keyGenerator: true,
+      ipv6Subnet: true,
+      handler: true,
+      skip: true,
+      requestWasSuccessful: true,
+      store: true,
+      validate: true,
+      headers: true,
+      max: true,
+      passOnStoreError: true
+    };
+    const validOptions = Object.keys(optionsMap).concat(
+      "draft_polli_ratelimit_headers",
+      // not a valid option anymore, but we have a more specific check for this one, so don't warn for it here
+      // from express-slow-down - https://github.com/express-rate-limit/express-slow-down/blob/main/source/types.ts#L65
+      "delayAfter",
+      "delayMs",
+      "maxDelayMs"
+    );
+    for (const key of Object.keys(passedOptions)) {
+      if (!validOptions.includes(key)) {
+        throw new ValidationError(
+          "ERR_ERL_UNKNOWN_OPTION",
+          `Unexpected configuration option: ${key}`
+          // todo: suggest a valid option with a short levenstein distance?
+        );
+      }
+    }
+  },
   /**
    * Checks the options.validate setting to ensure that only recognized
    * validations are enabled or disabled.
    *
    * If any unrecognized values are found, an error is logged that
-   * includes the list of supported vaidations.
+   * includes the list of supported validations.
    */
   validationsConfig() {
     const supportedValidations = Object.keys(this).filter(
@@ -4837,6 +4879,7 @@ var parseOptions = (passedOptions) => {
   const notUndefinedOptions = omitUndefinedProperties(passedOptions);
   const validations2 = getValidations(notUndefinedOptions?.validate ?? true);
   validations2.validationsConfig();
+  validations2.knownOptions(passedOptions);
   validations2.draftPolliHeaders(
     // @ts-expect-error see the note above.
     notUndefinedOptions.draft_polli_ratelimit_headers

package/dist/cli/server/server.js

@@ -4194,7 +4194,7 @@ var init_spreadsheetRoutes = __esm({
 import cors from "cors";
 import express2 from "express";
 
-// node_modules/.pnpm/express-rate-limit@8.1.0_express@5.1.0/node_modules/express-rate-limit/dist/index.mjs
+// node_modules/.pnpm/express-rate-limit@8.2.1_express@5.1.0/node_modules/express-rate-limit/dist/index.mjs
 var import_ip_address = __toESM(require_ip_address(), 1);
 import { isIPv6 } from "node:net";
 import { isIPv6 as isIPv62 } from "node:net";
@@ -4664,12 +4664,54 @@ var validations = {
       );
     }
   },
+  knownOptions(passedOptions) {
+    if (!passedOptions) return;
+    const optionsMap = {
+      windowMs: true,
+      limit: true,
+      message: true,
+      statusCode: true,
+      legacyHeaders: true,
+      standardHeaders: true,
+      identifier: true,
+      requestPropertyName: true,
+      skipFailedRequests: true,
+      skipSuccessfulRequests: true,
+      keyGenerator: true,
+      ipv6Subnet: true,
+      handler: true,
+      skip: true,
+      requestWasSuccessful: true,
+      store: true,
+      validate: true,
+      headers: true,
+      max: true,
+      passOnStoreError: true
+    };
+    const validOptions = Object.keys(optionsMap).concat(
+      "draft_polli_ratelimit_headers",
+      // not a valid option anymore, but we have a more specific check for this one, so don't warn for it here
+      // from express-slow-down - https://github.com/express-rate-limit/express-slow-down/blob/main/source/types.ts#L65
+      "delayAfter",
+      "delayMs",
+      "maxDelayMs"
+    );
+    for (const key of Object.keys(passedOptions)) {
+      if (!validOptions.includes(key)) {
+        throw new ValidationError(
+          "ERR_ERL_UNKNOWN_OPTION",
+          `Unexpected configuration option: ${key}`
+          // todo: suggest a valid option with a short levenstein distance?
+        );
+      }
+    }
+  },
   /**
    * Checks the options.validate setting to ensure that only recognized
    * validations are enabled or disabled.
    *
    * If any unrecognized values are found, an error is logged that
-   * includes the list of supported vaidations.
+   * includes the list of supported validations.
    */
   validationsConfig() {
     const supportedValidations = Object.keys(this).filter(
@@ -4837,6 +4879,7 @@ var parseOptions = (passedOptions) => {
   const notUndefinedOptions = omitUndefinedProperties(passedOptions);
   const validations2 = getValidations(notUndefinedOptions?.validate ?? true);
   validations2.validationsConfig();
+  validations2.knownOptions(passedOptions);
   validations2.draftPolliHeaders(
     // @ts-expect-error see the note above.
     notUndefinedOptions.draft_polli_ratelimit_headers

package/dist/index.html

@@ -6,28 +6,28 @@
 		<link rel="icon" href="/lula.png" />
 		<meta name="viewport" content="width=device-width, initial-scale=1" />
 		
-		<link rel="modulepreload" href="/_app/immutable/entry/start.BzhV34ug.js">
-		<link rel="modulepreload" href="/_app/immutable/chunks/l0xMBaDV.js">
-		<link rel="modulepreload" href="/_app/immutable/chunks/iLqChAUt.js">
-		<link rel="modulepreload" href="/_app/immutable/entry/app.DK674slU.js">
-		<link rel="modulepreload" href="/_app/immutable/chunks/B0ygo1VA.js">
-		<link rel="modulepreload" href="/_app/immutable/chunks/CpBmCwmc.js">
-		<link rel="modulepreload" href="/_app/immutable/chunks/P3psI8RV.js">
-		<link rel="modulepreload" href="/_app/immutable/chunks/iYE0hyoB.js">
+		<link rel="modulepreload" href="/_app/immutable/entry/start.Bbaig19m.js">
+		<link rel="modulepreload" href="/_app/immutable/chunks/Bjogn1-e.js">
+		<link rel="modulepreload" href="/_app/immutable/chunks/CKMdwa-C.js">
+		<link rel="modulepreload" href="/_app/immutable/entry/app.cmkEjIE0.js">
+		<link rel="modulepreload" href="/_app/immutable/chunks/DrUO72_y.js">
+		<link rel="modulepreload" href="/_app/immutable/chunks/Dtw5WU4b.js">
+		<link rel="modulepreload" href="/_app/immutable/chunks/CTPWKfKb.js">
+		<link rel="modulepreload" href="/_app/immutable/chunks/BpJCrXEQ.js">
 	</head>
 	<body data-sveltekit-preload-data="hover">
 		<div style="display: contents">
 			<script>
 				{
-					__sveltekit_prn659 = {
+					__sveltekit_1b0ggpf = {
 						base: ""
 					};
 
 					const element = document.currentScript.parentElement;
 
 					Promise.all([
-						import("/_app/immutable/entry/start.BzhV34ug.js"),
-						import("/_app/immutable/entry/app.DK674slU.js")
+						import("/_app/immutable/entry/start.Bbaig19m.js"),
+						import("/_app/immutable/entry/app.cmkEjIE0.js")
 					]).then(([kit, app]) => {
 						kit.start(app, element);
 					});

package/dist/index.js

@@ -4221,7 +4221,7 @@ import { join as join7 } from "path";
 import cors from "cors";
 import express2 from "express";
 
-// node_modules/.pnpm/express-rate-limit@8.1.0_express@5.1.0/node_modules/express-rate-limit/dist/index.mjs
+// node_modules/.pnpm/express-rate-limit@8.2.1_express@5.1.0/node_modules/express-rate-limit/dist/index.mjs
 var import_ip_address = __toESM(require_ip_address(), 1);
 import { isIPv6 } from "node:net";
 import { isIPv6 as isIPv62 } from "node:net";
@@ -4691,12 +4691,54 @@ var validations = {
       );
     }
   },
+  knownOptions(passedOptions) {
+    if (!passedOptions) return;
+    const optionsMap = {
+      windowMs: true,
+      limit: true,
+      message: true,
+      statusCode: true,
+      legacyHeaders: true,
+      standardHeaders: true,
+      identifier: true,
+      requestPropertyName: true,
+      skipFailedRequests: true,
+      skipSuccessfulRequests: true,
+      keyGenerator: true,
+      ipv6Subnet: true,
+      handler: true,
+      skip: true,
+      requestWasSuccessful: true,
+      store: true,
+      validate: true,
+      headers: true,
+      max: true,
+      passOnStoreError: true
+    };
+    const validOptions = Object.keys(optionsMap).concat(
+      "draft_polli_ratelimit_headers",
+      // not a valid option anymore, but we have a more specific check for this one, so don't warn for it here
+      // from express-slow-down - https://github.com/express-rate-limit/express-slow-down/blob/main/source/types.ts#L65
+      "delayAfter",
+      "delayMs",
+      "maxDelayMs"
+    );
+    for (const key of Object.keys(passedOptions)) {
+      if (!validOptions.includes(key)) {
+        throw new ValidationError(
+          "ERR_ERL_UNKNOWN_OPTION",
+          `Unexpected configuration option: ${key}`
+          // todo: suggest a valid option with a short levenstein distance?
+        );
+      }
+    }
+  },
   /**
    * Checks the options.validate setting to ensure that only recognized
    * validations are enabled or disabled.
    *
    * If any unrecognized values are found, an error is logged that
-   * includes the list of supported vaidations.
+   * includes the list of supported validations.
    */
   validationsConfig() {
     const supportedValidations = Object.keys(this).filter(
@@ -4864,6 +4906,7 @@ var parseOptions = (passedOptions) => {
   const notUndefinedOptions = omitUndefinedProperties(passedOptions);
   const validations2 = getValidations(notUndefinedOptions?.validate ?? true);
   validations2.validationsConfig();
+  validations2.knownOptions(passedOptions);
   validations2.draftPolliHeaders(
     // @ts-expect-error see the note above.
     notUndefinedOptions.draft_polli_ratelimit_headers
@@ -5920,17 +5963,22 @@ function getChangedBlocks(oldText, newText) {
   const oldBlocks = extractMapBlocks(oldText);
   const newBlocks = extractMapBlocks(newText);
   const changed = [];
+  const oldLines = oldText.split("\n");
+  const newLines = newText.split("\n");
   for (const newBlock of newBlocks) {
     const oldMatch = oldBlocks.find((b) => b.uuid === newBlock.uuid);
     if (!oldMatch) continue;
-    const oldSegment = oldText.split("\n").slice(oldMatch.startLine, oldMatch.endLine).join("\n");
-    const newSegment = newText.split("\n").slice(newBlock.startLine, newBlock.endLine).join("\n");
-    if (oldSegment !== newSegment) {
+    const oldContent = extractBlockContent(oldLines, oldMatch.startLine, oldMatch.endLine);
+    const newContent = extractBlockContent(newLines, newBlock.startLine, newBlock.endLine);
+    if (oldContent !== newContent) {
       changed.push(newBlock);
     }
   }
   return changed;
 }
+function extractBlockContent(lines, startLine, endLine) {
+  return lines.slice(startLine + 1, endLine - 1).join("\n");
+}
 function getRemovedBlocks(oldText, newText) {
   const oldBlocks = extractMapBlocks(oldText);
   const newBlocks = extractMapBlocks(newText);
@@ -6011,20 +6059,27 @@ Please review whether:
   return { hasFindings: true, warningContent };
 }
 function generateChangedBlocksContent(filename, changedBlocks, newText) {
-  let content = "";
-  for (const block of changedBlocks) {
-    console.log(`Commenting regarding \`${filename}\`.`);
-    content += `
+  if (changedBlocks.length === 0) {
+    return "";
+  }
+  console.log(`Commenting regarding \`${filename}\`.`);
+  let content = `
 
 ---
 | File | Lines Changed |
 | ---- | ------------- |
 `;
+  for (const block of changedBlocks) {
+    content += `| \`${filename}\` | \`${block.startLine + 1}\u2013${block.endLine}\` |
+`;
+  }
+  content += `
+`;
+  for (const block of changedBlocks) {
     const newBlockText = newText.split("\n").slice(block.startLine, block.endLine).join("\n");
     const blockSha256 = createHash2("sha256").update(newBlockText).digest("hex");
-    content += `| \`${filename}\` | \`${block.startLine + 1}\u2013${block.endLine}\` |
-> **uuid**-\`${block.uuid}\`
- **sha256** \`${blockSha256}\`
+    content += `**UUID:** \`${block.uuid}\`
+**sha256:** \`${blockSha256}\`
 
 `;
   }
@@ -6048,11 +6103,16 @@ function generateRemovedBlocksContent(filename, removedBlocks, oldText) {
   content += `| ---- | -------------- | ---- |
 `;
   for (const block of removedBlocks) {
-    const oldBlockText = oldText.split("\n").slice(block.startLine, block.endLine).join("\n");
-    const blockSha256 = createHash2("sha256").update(oldBlockText).digest("hex");
     content += `| \`${filename}\` | \`${block.startLine + 1}\u2013${block.endLine}\` | \`${block.uuid}\` |
 `;
-    content += `> **sha256** \`${blockSha256}\`
+  }
+  content += `
+`;
+  for (const block of removedBlocks) {
+    const oldBlockText = oldText.split("\n").slice(block.startLine, block.endLine).join("\n");
+    const blockSha256 = createHash2("sha256").update(oldBlockText).digest("hex");
+    content += `**UUID:** \`${block.uuid}\`
+**sha256:** \`${blockSha256}\`
 
 `;
   }

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "lula2",
-  "version": "0.7.0",
+  "version": "0.7.1-nightly.0",
   "description": "A tool for managing compliance as code in your GitHub repositories.",
   "bin": {
     "lula2": "./dist/lula2"
@@ -33,25 +33,6 @@
     "!dist/**/*.test.js*",
     "!dist/**/*.test.d.ts*"
   ],
-  "scripts": {
-    "dev": "vite dev --port 5173",
-    "dev:api": "tsx --watch index.ts --debug ui --port 3000 --no-open-browser",
-    "dev:full": "concurrently \"npm run dev:api\" \"npm run dev\"",
-    "build": "npm run build:svelte && npm run build:cli && npm run postbuild:cli",
-    "build:svelte": "vite build",
-    "build:cli": "esbuild index.ts cli/**/*.ts --bundle --platform=node --target=node22 --format=esm --outdir=dist --external:express --external:commander --external:js-yaml --external:yaml --external:isomorphic-git --external:glob --external:open --external:ws --external:cors --external:multer --external:@octokit/rest --external:undici --external:xlsx-republish --external:csv-parse",
-    "postbuild:cli": "cp cli-wrapper.mjs dist/lula2 && chmod +x dist/lula2",
-    "preview": "vite preview",
-    "prepare": "svelte-kit sync || echo ''",
-    "check": "svelte-kit sync && svelte-check --tsconfig ./tsconfig.json && tsc --noEmit",
-    "check:watch": "svelte-kit sync && svelte-check --tsconfig ./tsconfig.json --watch",
-    "format": "prettier --write 'src/**/*.{ts,js,svelte}' 'cli/**/*.ts' 'index.ts' 'tests/**/*.ts'",
-    "format:check": "prettier --check 'src/**/*.{ts,js,svelte}' 'cli/**/*.ts' 'index.ts' 'tests/**/*.ts'",
-    "lint": "prettier --check 'src/**/*.{ts,js,svelte}' 'cli/**/*.ts' 'index.ts' 'tests/**/*.ts' && eslint src cli",
-    "test": "npm run test:unit -- --run --coverage",
-    "test:integration": "vitest --config integration/vitest.config.integration.ts",
-    "test:unit": "vitest"
-  },
   "dependencies": {
     "@octokit/rest": "^22.0.0",
     "@types/ws": "^8.18.1",
@@ -95,7 +76,7 @@
     "carbon-icons-svelte": "^13.5.0",
     "carbon-preprocess-svelte": "^0.11.11",
     "concurrently": "^9.2.1",
-    "esbuild": "^0.25.9",
+    "esbuild": "^0.27.0",
     "eslint": "^9.35.0",
     "eslint-config-prettier": "^10.1.8",
     "eslint-plugin-jsdoc": "^61.0.0",
@@ -124,5 +105,23 @@
       "main",
       "next"
     ]
+  },
+  "scripts": {
+    "dev": "vite dev --port 5173",
+    "dev:api": "tsx --watch index.ts --debug ui --port 3000 --no-open-browser",
+    "dev:full": "concurrently \"npm run dev:api\" \"npm run dev\"",
+    "build": "npm run build:svelte && npm run build:cli && npm run postbuild:cli",
+    "build:svelte": "vite build",
+    "build:cli": "esbuild index.ts cli/**/*.ts --bundle --platform=node --target=node22 --format=esm --outdir=dist --external:express --external:commander --external:js-yaml --external:yaml --external:isomorphic-git --external:glob --external:open --external:ws --external:cors --external:multer --external:@octokit/rest --external:undici --external:xlsx-republish --external:csv-parse",
+    "postbuild:cli": "cp cli-wrapper.mjs dist/lula2 && chmod +x dist/lula2",
+    "preview": "vite preview",
+    "check": "svelte-kit sync && svelte-check --tsconfig ./tsconfig.json && tsc --noEmit",
+    "check:watch": "svelte-kit sync && svelte-check --tsconfig ./tsconfig.json --watch",
+    "format": "prettier --write 'src/**/*.{ts,js,svelte}' 'cli/**/*.ts' 'index.ts' 'tests/**/*.ts'",
+    "format:check": "prettier --check 'src/**/*.{ts,js,svelte}' 'cli/**/*.ts' 'index.ts' 'tests/**/*.ts'",
+    "lint": "prettier --check 'src/**/*.{ts,js,svelte}' 'cli/**/*.ts' 'index.ts' 'tests/**/*.ts' && eslint src cli",
+    "test": "npm run test:unit -- --run --coverage",
+    "test:integration": "vitest --config integration/vitest.config.integration.ts",
+    "test:unit": "vitest"
   }
-}
+}