ci: harden the desktop changeset linker

Accept quoted bump values, keep the CLI entry's indentation on the
inserted desktop line, and resolve the main-module guard through
pathToFileURL so a path with URL characters still runs the linker.

Author: elkaix

scripts/release/link-desktop-changesets.mjs

@@ -15,11 +15,12 @@
 
 import { readdirSync, readFileSync, writeFileSync } from 'node:fs';
 import { join } from 'node:path';
+import { pathToFileURL } from 'node:url';
 
 export const CLI_PACKAGE = '@pymodel/pythinker-code';
 export const DESKTOP_PACKAGE = '@pymodel/pythinker-desktop';
 
-const FRONTMATTER_LINE = /^(\s*)(?:"([^"]+)"|'([^']+)'|([^:'"]+?))\s*:\s*([A-Za-z]+)\s*$/u;
+const FRONTMATTER_LINE = /^(\s*)(?:"([^"]+)"|'([^']+)'|([^:'"]+?))\s*:\s*(?:"([A-Za-z]+)"|'([A-Za-z]+)'|([A-Za-z]+))\s*$/u;
 
 /**
  * Add the desktop package to one changeset when it names the CLI alone.
@@ -36,19 +37,21 @@ export function linkDesktop(source) {
 
   let cliLine = -1;
   let cliLevel = null;
+  let cliIndent = '';
   for (const [index, line] of lines.entries()) {
     const match = FRONTMATTER_LINE.exec(line);
     if (match === null) continue;
     const name = match[2] ?? match[3] ?? match[4];
     if (name === DESKTOP_PACKAGE) return null;
     if (name === CLI_PACKAGE) {
       cliLine = index;
-      cliLevel = match[5].toLowerCase();
+      cliLevel = (match[5] ?? match[6] ?? match[7]).toLowerCase();
+      cliIndent = match[1];
     }
   }
   if (cliLine === -1) return null;
 
-  lines.splice(cliLine + 1, 0, `"${DESKTOP_PACKAGE}": ${cliLevel}`);
+  lines.splice(cliLine + 1, 0, `${cliIndent}"${DESKTOP_PACKAGE}": ${cliLevel}`);
   return `---\n${lines.join('\n')}${normalized.slice(end + 1)}`;
 }
 
@@ -71,7 +74,7 @@ export function linkDesktopChangesets(dir) {
   return changed;
 }
 
-if (process.argv[1] !== undefined && import.meta.url === new URL(`file://${process.argv[1]}`).href) {
+if (process.argv[1] !== undefined && import.meta.url === pathToFileURL(process.argv[1]).href) {
   const changed = linkDesktopChangesets(process.argv[2] ?? '.changeset');
   for (const name of changed) console.log(`linked desktop bump: ${name}`);
   console.log(`${changed.length} changeset(s) now bump ${DESKTOP_PACKAGE} alongside ${CLI_PACKAGE}.`);

scripts/release/link-desktop-changesets.test.mjs

@@ -21,6 +21,20 @@ void test('leaves a changeset that already names desktop alone', () => {
   assert.equal(linkDesktop(linkDesktop(cliOnly)), null);
 });
 
+void test('accepts quoted bump values and does not duplicate the desktop entry', () => {
+  const quoted = '---\n"@pymodel/pythinker-code": "minor"\n---\n\nx\n';
+  assert.equal(linkDesktop(quoted), '---\n"@pymodel/pythinker-code": "minor"\n"@pymodel/pythinker-desktop": minor\n---\n\nx\n');
+  assert.equal(linkDesktop(linkDesktop(quoted)), null);
+  assert.equal(linkDesktop("---\n'@pymodel/pythinker-desktop': 'patch'\n\"@pymodel/pythinker-code\": 'patch'\n---\n\nx\n"), null);
+});
+
+void test('reuses the CLI entry indentation', () => {
+  assert.equal(
+    linkDesktop('---\n  "@pymodel/pythinker-code": patch\n---\n\nx\n'),
+    '---\n  "@pymodel/pythinker-code": patch\n  "@pymodel/pythinker-desktop": patch\n---\n\nx\n',
+  );
+});
+
 void test('ignores changesets that do not name the CLI', () => {
   assert.equal(linkDesktop('---\n"@pymodel/klient": patch\n---\n\nx\n'), null);
   assert.equal(linkDesktop('no frontmatter\n'), null);