Address review feedback: schema version, node engine, repo URL parsing

Co-authored-by: TylerJDev <26746305+TylerJDev@users.noreply.github.com>

Author: Copilot

.changeset/config.json

@@ -1,5 +1,5 @@
 {
-  "$schema": "https://unpkg.com/@changesets/config@3.0.0/schema.json",
+  "$schema": "https://unpkg.com/@changesets/config@4.0.0/schema.json",
   "changelog": ["@changesets/changelog-github", {"repo": "github/remote-input-element"}],
   "commit": false,
   "fixed": [],

.github/workflows/release.yml

@@ -25,7 +25,7 @@ jobs:
 
       - uses: actions/setup-node@v4
         with:
-          node-version: 18
+          node-version: 22
           registry-url: https://registry.npmjs.org
 
       - name: Install dependencies

scripts/prepare-release.mjs

@@ -80,6 +80,30 @@ export function getCommitsSinceTag(range, cwd, repository) {
   return entries
 }
 
+// Normalizes the `repository` field of package.json (which may be a plain
+// "owner/repo" shorthand string, a full git/https URL string, or an object
+// with a `url` property) into a plain "owner/repo" string suitable for
+// building GitHub URLs.
+export function normalizeRepository(repository) {
+  const raw = typeof repository === 'string' ? repository : repository?.url
+  if (!raw) {
+    throw new Error('Unable to determine the GitHub "owner/repo" from package.json\'s "repository" field.')
+  }
+
+  // Plain "owner/repo" shorthand, e.g. "github/remote-input-element".
+  if (/^[^/\s:]+\/[^/\s]+$/.test(raw)) {
+    return raw
+  }
+
+  // A GitHub URL (git/https/ssh), e.g.
+  // "git+https://github.com/owner/repo.git" or "git@github.com:owner/repo.git".
+  const match = raw.match(/(?:^|\/\/|@)github\.com[:/]([^/\s]+\/[^/\s]+?)(?:\.git)?\/?$/)
+  if (!match) {
+    throw new Error(`Unable to parse a GitHub "owner/repo" from repository field: ${JSON.stringify(raw)}`)
+  }
+  return match[1]
+}
+
 function parseCommit({sha, subject, body}, repository) {
   const shortSha = sha.slice(0, 7)
   const mergeMatch = subject.match(/^Merge pull request #(\d+) from/)
@@ -151,15 +175,16 @@ export function prepareRelease({cwd = process.cwd()} = {}) {
   }
 
   const range = `${tag}..HEAD`
-  const entries = getCommitsSinceTag(range, cwd, pkg.repository)
+  const repository = normalizeRepository(pkg.repository)
+  const entries = getCommitsSinceTag(range, cwd, repository)
   if (entries.length === 0) {
     if (existsSync(autoChangesetPath)) {
       rmSync(autoChangesetPath)
     }
     return {action: 'skipped-no-changes', tag}
   }
 
-  const content = buildChangesetContent(pkg.name, entries, pkg.repository)
+  const content = buildChangesetContent(pkg.name, entries, repository)
   writeFileSync(autoChangesetPath, content)
   return {action: 'created', tag, path: autoChangesetPath, entries}
 }

scripts/prepare-release.test.mjs

@@ -5,7 +5,7 @@ import os from 'node:os'
 import path from 'node:path'
 import {test, describe, beforeEach, afterEach} from 'node:test'
 
-import {prepareRelease, AUTO_CHANGESET_FILENAME} from './prepare-release.mjs'
+import {prepareRelease, AUTO_CHANGESET_FILENAME, normalizeRepository} from './prepare-release.mjs'
 
 function git(args, cwd) {
   return execFileSync('git', args, {cwd, encoding: 'utf8'}).trim()
@@ -41,6 +41,30 @@ afterEach(() => {
   rmSync(repoDir, {recursive: true, force: true})
 })
 
+describe('normalizeRepository', () => {
+  test('accepts the "owner/repo" shorthand string', () => {
+    assert.equal(normalizeRepository('github/remote-input-element'), 'github/remote-input-element')
+  })
+
+  test('accepts a full git URL string', () => {
+    assert.equal(
+      normalizeRepository('git+https://github.com/github/remote-input-element.git'),
+      'github/remote-input-element',
+    )
+  })
+
+  test('accepts an object with a url property', () => {
+    assert.equal(
+      normalizeRepository({type: 'git', url: 'https://github.com/github/remote-input-element.git'}),
+      'github/remote-input-element',
+    )
+  })
+
+  test('rejects a non-GitHub URL instead of extracting the wrong owner/repo', () => {
+    assert.throws(() => normalizeRepository('https://notgithub.com/owner/repo'))
+  })
+})
+
 describe('prepareRelease', () => {
   test('does nothing when the checked-in version has no matching tag (pending publication)', () => {
     writePackageJson(repoDir, '0.4.0')