chore: bump char limit, add function to create URL

Author: bombillazo

src/createIssue.ts

@@ -8,9 +8,16 @@ import { join, resolve } from "./path"
 const repoSpecifier = /^([\w.-]+)\/([\w.-]+)$/
 const githubURL = /github.com(:|\/)([\w.-]+\/[\w.-]+?)(.git|\/.*)?$/
 
-function parseRepoString(
-  repository: string,
-): null | { repo: string; org: string; provider: "GitHub" } {
+type VCS =
+  | {
+      repo: string
+      org: string
+      provider: "GitHub"
+    }
+  | null
+  | undefined
+
+function parseRepoString(repository: string): VCS {
   if (repository.startsWith("github:")) {
     repository = repository.replace(/^github:/, "")
   }
@@ -29,7 +36,7 @@ function parseRepoString(
   return { org, repo, provider: "GitHub" }
 }
 
-function getPackageVCSDetails(packageDetails: PackageDetails) {
+function getPackageVCSDetails(packageDetails: PackageDetails): VCS {
   const repository = require(resolve(join(packageDetails.path, "package.json")))
     .repository as undefined | string | { url: string }
 
@@ -46,6 +53,38 @@ function getPackageVCSDetails(packageDetails: PackageDetails) {
   }
 }
 
+function createIssueUrl({
+  vcs,
+  packageDetails,
+  packageVersion,
+  diff,
+}: {
+  vcs: VCS
+  packageDetails: PackageDetails
+  packageVersion: string
+  diff: string
+}): string {
+  return `https://github.com/${vcs?.org}/${vcs?.repo}/issues/new?${stringify({
+    title: "",
+    body: `Hi! 👋 
+      
+Firstly, thanks for your work on this project! 🙂
+
+Today I used [patch-package](https://github.com/ds300/patch-package) to patch \`${packageDetails.name}@${packageVersion}\` for the project I'm working on.
+
+<!-- 🔺️🔺️🔺️ PLEASE REPLACE THIS BLOCK with a description of your problem, and any other relevant context 🔺️🔺️🔺️ -->
+
+Here is the diff that solved my problem:
+
+\`\`\`diff
+${diff}
+\`\`\`
+
+<em>This issue body was [partially generated by patch-package](https://github.com/ds300/patch-package/issues/296).</em>
+`,
+  })}`
+}
+
 export function shouldRecommendIssue(
   vcsDetails: ReturnType<typeof getPackageVCSDetails>,
 ) {
@@ -99,36 +138,28 @@ export function openIssueCreationLink({
   if (patchFileContents.endsWith("\n")) {
     patchFileContents = patchFileContents.slice(0, -1)
   }
-  
-  const patchContentExceedsLimit = patchFileContents.length > 300;
-  let diffContents = patchFileContents;
-  
-  if (patchContentExceedsLimit) {
-    diffContents = '<!-- 🔺️🔺️🔺️ PLEASE REPLACE THIS BLOCK with the diff contents printed out by the `--create-issue` command. 🔺️🔺️🔺️ -->';
-    console.log(`📋 Copy the following diff contents and paste it in the new issue's diff section:
-    
-${patchFileContents}
-`);
-  }
-  open(
-    `https://github.com/${vcs.org}/${vcs.repo}/issues/new?${stringify({
-      title: "",
-      body: `Hi! 👋 
-      
-Firstly, thanks for your work on this project! 🙂
 
-Today I used [patch-package](https://github.com/ds300/patch-package) to patch \`${packageDetails.name}@${packageVersion}\` for the project I'm working on.
+  let issueUrl = createIssueUrl({
+    vcs,
+    packageDetails,
+    packageVersion,
+    diff: patchFileContents,
+  })
 
-<!-- 🔺️🔺️🔺️ PLEASE REPLACE THIS BLOCK with a description of your problem, and any other relevant context 🔺️🔺️🔺️ -->
+  const urlExceedsLimit = patchFileContents.length > 1950
 
-Here is the diff that solved my problem:
-
-\`\`\`diff
-${diffContents}
-\`\`\`
-
-<em>This issue body was [partially generated by patch-package](https://github.com/ds300/patch-package/issues/296).</em>
-`,
-    })}`,
-  )
+  if (urlExceedsLimit) {
+    const diffMessage =
+      "<!-- 🔺️🔺️🔺️ PLEASE REPLACE THIS BLOCK with the diff contents of . 🔺️🔺️🔺️ -->"
+    console.log(
+      `📋 Copy the contents in and paste it in the new issue's diff section:`,
+    )
+    issueUrl = createIssueUrl({
+      vcs,
+      packageDetails,
+      packageVersion,
+      diff: diffMessage,
+    })
+  }
+  open(issueUrl)
 }