skilld 0.10.0 → 0.10.2

package/dist/_chunks/npm.mjs

@@ -116,7 +116,7 @@ function hasCodeBlock$1(text) {
 function isNonTechnical(issue) {
 	const body = (issue.body || "").trim();
 	if (body.length < 200 && !hasCodeBlock$1(body) && issue.reactions > 50) return true;
-	if (/\b(?:roadmap|tracking|love|thank|awesome|great work)\b/i.test(issue.title) && !hasCodeBlock$1(body)) return true;
+	if (/\b(?:love|thank|awesome|great work)\b/i.test(issue.title) && !hasCodeBlock$1(body)) return true;
 	return false;
 }
 function freshnessScore(reactions, createdAt) {
@@ -197,18 +197,26 @@ function fetchIssuesByState(owner, repo, state, count, releasedAt, fromDate) {
 		"api",
 		`search/issues?q=${`repo:${owner}/${repo}+is:issue+is:${state}${datePart}`}&sort=reactions&order=desc&per_page=${fetchCount}`,
 		"-q",
-		".items[] | {number, title, state, labels: [.labels[]?.name], body, createdAt: .created_at, url: .html_url, reactions: .reactions[\"+1\"], comments: .comments, user: .user.login, userType: .user.type}"
+		".items[] | {number, title, state, labels: [.labels[]?.name], body, createdAt: .created_at, url: .html_url, reactions: .reactions[\"+1\"], comments: .comments, user: .user.login, userType: .user.type, authorAssociation: .author_association}"
 	], {
 		encoding: "utf-8",
 		maxBuffer: 10 * 1024 * 1024
 	});
 	if (!result) return [];
-	return result.trim().split("\n").filter(Boolean).map((line) => JSON.parse(line)).filter((issue) => !BOT_USERS.has(issue.user) && issue.userType !== "Bot").filter((issue) => !isNoiseIssue(issue)).filter((issue) => !isNonTechnical(issue)).map(({ user: _, userType: __, ...issue }) => ({
-		...issue,
-		type: classifyIssue(issue.labels),
-		topComments: [],
-		score: freshnessScore(issue.reactions, issue.createdAt)
-	})).sort((a, b) => b.score - a.score).slice(0, count);
+	return result.trim().split("\n").filter(Boolean).map((line) => JSON.parse(line)).filter((issue) => !BOT_USERS.has(issue.user) && issue.userType !== "Bot").filter((issue) => !isNoiseIssue(issue)).filter((issue) => !isNonTechnical(issue)).map(({ user: _, userType: __, authorAssociation, ...issue }) => {
+		const isMaintainer = [
+			"OWNER",
+			"MEMBER",
+			"COLLABORATOR"
+		].includes(authorAssociation);
+		const isRoadmap = /\broadmap\b/i.test(issue.title) || issue.labels.some((l) => /roadmap/i.test(l));
+		return {
+			...issue,
+			type: classifyIssue(issue.labels),
+			topComments: [],
+			score: freshnessScore(issue.reactions, issue.createdAt) * (isMaintainer && isRoadmap ? 5 : 1)
+		};
+	}).sort((a, b) => b.score - a.score).slice(0, count);
 }
 function oneYearAgo() {
 	const d = /* @__PURE__ */ new Date();
@@ -480,22 +488,23 @@ function compareSemver(a, b) {
 }
 function fetchReleasesViaGh(owner, repo) {
 	try {
-		const { stdout: json } = spawnSync("gh", [
+		const { stdout: ndjson } = spawnSync("gh", [
 			"api",
-			`repos/${owner}/${repo}/releases?per_page=100`,
+			`repos/${owner}/${repo}/releases`,
+			"--paginate",
 			"--jq",
-			"[.[] | {id: .id, tag: .tag_name, name: .name, prerelease: .prerelease, createdAt: .created_at, publishedAt: .published_at, markdown: .body}]"
+			".[] | {id: .id, tag: .tag_name, name: .name, prerelease: .prerelease, createdAt: .created_at, publishedAt: .published_at, markdown: .body}"
 		], {
 			encoding: "utf-8",
-			timeout: 15e3,
+			timeout: 3e4,
 			stdio: [
 				"ignore",
 				"pipe",
 				"ignore"
 			]
 		});
-		if (!json) return [];
-		return JSON.parse(json);
+		if (!ndjson) return [];
+		return ndjson.trim().split("\n").filter(Boolean).map((line) => JSON.parse(line));
 	} catch {
 		return [];
 	}
@@ -597,11 +606,6 @@ function isStubRelease(release) {
 	const body = (release.markdown || "").trim();
 	return body.length < 500 && /changelog\.md/i.test(body);
 }
-function isChangelogRedirectPattern(releases) {
-	const sample = releases.slice(0, 3);
-	if (sample.length === 0) return false;
-	return sample.every(isStubRelease);
-}
 async function fetchChangelog(owner, repo, ref, packageName) {
 	const paths = [];
 	if (packageName) {
@@ -620,30 +624,23 @@ async function fetchChangelog(owner, repo, ref, packageName) {
 	}
 	return null;
 }
-async function fetchReleaseNotes(owner, repo, installedVersion, gitRef, packageName, fromDate) {
+async function fetchReleaseNotes(owner, repo, installedVersion, gitRef, packageName, fromDate, changelogRef) {
 	const selected = selectReleases(await fetchAllReleases(owner, repo), packageName, installedVersion, fromDate);
 	if (selected.length > 0) {
-		if (isChangelogRedirectPattern(selected)) {
-			const changelog = await fetchChangelog(owner, repo, gitRef || selected[0].tag, packageName);
-			if (changelog) return [{
-				path: "releases/CHANGELOG.md",
-				content: changelog
-			}];
-		}
 		const docs = selected.filter((r) => !isStubRelease(r)).map((r) => {
 			return {
 				path: `releases/${r.tag.includes("@") || r.tag.startsWith("v") ? r.tag : `v${r.tag}`}.md`,
 				content: formatRelease(r, packageName)
 			};
 		});
-		const changelog = await fetchChangelog(owner, repo, gitRef || selected[0].tag, packageName);
+		const changelog = await fetchChangelog(owner, repo, changelogRef || gitRef || selected[0].tag, packageName);
 		if (changelog && changelog.length < 5e5) docs.push({
 			path: "releases/CHANGELOG.md",
 			content: changelog
 		});
 		return docs;
 	}
-	const changelog = await fetchChangelog(owner, repo, gitRef || "main", packageName);
+	const changelog = await fetchChangelog(owner, repo, changelogRef || gitRef || "main", packageName);
 	if (!changelog) return [];
 	return [{
 		path: "releases/CHANGELOG.md",
@@ -758,9 +755,29 @@ function truncateBody(body, limit) {
 	if (lastParagraph > lastSafeEnd * .6) return `${slice.slice(0, lastParagraph)}\n\n...`;
 	return `${slice}...`;
 }
+const TITLE_NOISE_RE = /looking .*(developer|engineer|freelanc)|hiring|job post|guide me to (?:complete|finish|build)|help me (?:complete|finish|build)|seeking .* tutorial|recommend.* course/i;
+const MIN_DISCUSSION_SCORE = 3;
 function scoreComment(c) {
 	return (c.isMaintainer ? 3 : 1) * (hasCodeBlock(c.body) ? 2 : 1) * (1 + c.reactions);
 }
+function scoreDiscussion(d) {
+	if (TITLE_NOISE_RE.test(d.title)) return -1;
+	let score = 0;
+	if (d.isMaintainer) score += 3;
+	if (hasCodeBlock([
+		d.body,
+		d.answer || "",
+		...d.topComments.map((c) => c.body)
+	].join("\n"))) score += 3;
+	score += Math.min(d.upvoteCount, 5);
+	if (d.answer) {
+		score += 2;
+		if (d.answer.length > 100) score += 1;
+	}
+	if (d.topComments.some((c) => c.isMaintainer)) score += 2;
+	if (d.topComments.some((c) => c.reactions > 0)) score += 1;
+	return score;
+}
 async function fetchGitHubDiscussions(owner, repo, limit = 20, releasedAt, fromDate) {
 	if (!isGhAvailable()) return [];
 	if (!fromDate && releasedAt) {
@@ -773,7 +790,7 @@ async function fetchGitHubDiscussions(owner, repo, limit = 20, releasedAt, fromD
 			"api",
 			"graphql",
 			"-f",
-			`query=${`query($owner: String!, $repo: String!) { repository(owner: $owner, name: $repo) { discussions(first: ${Math.min(limit * 3, 80)}, orderBy: {field: CREATED_AT, direction: DESC}) { nodes { number title body category { name } createdAt url upvoteCount comments(first: 10) { totalCount nodes { body author { login } authorAssociation reactions { totalCount } } } answer { body author { login } authorAssociation } author { login } } } } }`}`,
+			`query=${`query($owner: String!, $repo: String!) { repository(owner: $owner, name: $repo) { discussions(first: ${Math.min(limit * 3, 80)}, orderBy: {field: CREATED_AT, direction: DESC}) { nodes { number title body category { name } createdAt url upvoteCount comments(first: 10) { totalCount nodes { body author { login } authorAssociation reactions { totalCount } } } answer { body author { login } authorAssociation } author { login } authorAssociation } } } }`}`,
 			"-f",
 			`owner=${owner}`,
 			"-f",
@@ -822,15 +839,23 @@ async function fetchGitHubDiscussions(owner, repo, limit = 20, releasedAt, fromD
 				url: d.url,
 				upvoteCount: d.upvoteCount || 0,
 				comments: d.comments?.totalCount || 0,
+				isMaintainer: [
+					"OWNER",
+					"MEMBER",
+					"COLLABORATOR"
+				].includes(d.authorAssociation),
 				answer,
 				topComments: comments
 			};
-		}).sort((a, b) => {
-			const aHigh = HIGH_VALUE_CATEGORIES.has(a.category.toLowerCase()) ? 1 : 0;
-			const bHigh = HIGH_VALUE_CATEGORIES.has(b.category.toLowerCase()) ? 1 : 0;
+		}).map((d) => ({
+			d,
+			score: scoreDiscussion(d)
+		})).filter(({ score }) => score >= MIN_DISCUSSION_SCORE).sort((a, b) => {
+			const aHigh = HIGH_VALUE_CATEGORIES.has(a.d.category.toLowerCase()) ? 1 : 0;
+			const bHigh = HIGH_VALUE_CATEGORIES.has(b.d.category.toLowerCase()) ? 1 : 0;
 			if (aHigh !== bHigh) return bHigh - aHigh;
-			return b.upvoteCount + b.comments - (a.upvoteCount + a.comments);
-		}).slice(0, limit);
+			return b.score - a.score;
+		}).slice(0, limit).map(({ d }) => d);
 	} catch {
 		return [];
 	}