@mastra/github-signals 0.2.5 → 0.3.0

package/CHANGELOG.md

@@ -1,5 +1,35 @@
 # @mastra/github-signals
 
+## 0.3.0
+
+### Minor Changes
+
+- Added intent-aware GitHub pull request subscription modes. Review mode follows code revisions, authorized comments, review-thread state, and terminal PR state without CI or mergeability noise, while omitted modes retain working behavior. ([#21542](https://github.com/mastra-ai/mastra/pull/21542))
+
+  ```ts
+  await githubSignals.subscribeThreadToPR({ threadId, resourceId, pr, mode: 'review' });
+  ```
+
+### Patch Changes
+
+- Updated dependencies [[`79f04a7`](https://github.com/mastra-ai/mastra/commit/79f04a7f6c6829da541139f638f2f1d267916e08), [`65edab1`](https://github.com/mastra-ai/mastra/commit/65edab1c233d17b8f163bad12fca410d0e6f16b1), [`1e47b75`](https://github.com/mastra-ai/mastra/commit/1e47b7520cab4cfaa8daed52f17e2e6d14ff7539), [`ab20a38`](https://github.com/mastra-ai/mastra/commit/ab20a38d0275f8d85e0f3833bd87ef487bcc609f), [`fd4d5fe`](https://github.com/mastra-ai/mastra/commit/fd4d5fe4f943699b85db5e74404f190d5a6b8c2a), [`ae8790c`](https://github.com/mastra-ai/mastra/commit/ae8790c4bfaa088d2ab279d1dcc06f326b9fd109), [`2c85f42`](https://github.com/mastra-ai/mastra/commit/2c85f428e04ccd63ea31a7ec80b5b327afdad555), [`11bbeb9`](https://github.com/mastra-ai/mastra/commit/11bbeb9b108ef2264e05acefc6dafb9cbb342921), [`48ef1f1`](https://github.com/mastra-ai/mastra/commit/48ef1f1d24eedafbb07f64e659a81b52b67b8bf6), [`aa3a85d`](https://github.com/mastra-ai/mastra/commit/aa3a85daf094c683bb97efdf4b6a696d2e474af5), [`d29d06f`](https://github.com/mastra-ai/mastra/commit/d29d06fe00bbd35b4571150ea04c59d2ed783c71), [`e6516df`](https://github.com/mastra-ai/mastra/commit/e6516dfcdae4f4ac0e7971d84359a81385ee602f), [`1a485f3`](https://github.com/mastra-ai/mastra/commit/1a485f3538f5ec64d58bd8b5e1e99de0c695c87b), [`0d37487`](https://github.com/mastra-ai/mastra/commit/0d37487d9f349388a3f1cef6a536cf9dcc4b6273), [`8661d7d`](https://github.com/mastra-ai/mastra/commit/8661d7d7179f0a024456aabdd8679bcecd09ac28), [`dbbfeb8`](https://github.com/mastra-ai/mastra/commit/dbbfeb85ec949dc9ebc0755e1ad262e4f5eba8db), [`575e343`](https://github.com/mastra-ai/mastra/commit/575e343900451021d96110916497d334af7bc252), [`0b2a3d1`](https://github.com/mastra-ai/mastra/commit/0b2a3d1783875c5b97b7b36ab3d03d7360e0dde7), [`6bb5d71`](https://github.com/mastra-ai/mastra/commit/6bb5d7193fe9166b219f0fccae17db7a5ae86e65), [`3cc9d00`](https://github.com/mastra-ai/mastra/commit/3cc9d00b2b4333e0377a5e9df5eff92c17ce7630), [`cacb839`](https://github.com/mastra-ai/mastra/commit/cacb8392d9e74189b56d857290b0615f98a2683d), [`57de7d6`](https://github.com/mastra-ai/mastra/commit/57de7d644ba7146edb4e9e6111ec4fa98c3a59e9), [`c8e4cea`](https://github.com/mastra-ai/mastra/commit/c8e4ceac9a390d78c8327dff3cdb2861dd71957f), [`ed01e9a`](https://github.com/mastra-ai/mastra/commit/ed01e9a807514a904374bf687a7b8f18750f6f78), [`b47b26e`](https://github.com/mastra-ai/mastra/commit/b47b26e6fe95cb8a3482be2c5e52de157fe59d0b), [`0d37487`](https://github.com/mastra-ai/mastra/commit/0d37487d9f349388a3f1cef6a536cf9dcc4b6273), [`733a537`](https://github.com/mastra-ai/mastra/commit/733a537489a858b5880b2e98809334fba895a221), [`e8e299c`](https://github.com/mastra-ai/mastra/commit/e8e299cc6abdfc39947e2fec25803493015d3882), [`edfc548`](https://github.com/mastra-ai/mastra/commit/edfc548886bc7bae17b681f8b6b41a47eb32bcd2), [`b05f486`](https://github.com/mastra-ai/mastra/commit/b05f48612984d5fe2447ea2d6cdd5c604d285b97), [`a8a4871`](https://github.com/mastra-ai/mastra/commit/a8a4871215f51da95c47129602157ce5372f634a), [`eb9ecaa`](https://github.com/mastra-ai/mastra/commit/eb9ecaa89c36e889749e3b825cfc507ce7f7980b), [`4ff3ee2`](https://github.com/mastra-ai/mastra/commit/4ff3ee2bff7ed07528b4817f8f49639031c72a4d), [`9207dfa`](https://github.com/mastra-ai/mastra/commit/9207dfab8062e5fc68b751684797ff86fe0b4e70), [`5165cdc`](https://github.com/mastra-ai/mastra/commit/5165cdcdcf50e144bb8113278535196cc9b07065), [`e737014`](https://github.com/mastra-ai/mastra/commit/e737014e0fc7035759762bb5b48baef1d6c0f6a7), [`6bb5d71`](https://github.com/mastra-ai/mastra/commit/6bb5d7193fe9166b219f0fccae17db7a5ae86e65), [`f591643`](https://github.com/mastra-ai/mastra/commit/f591643becdf0be9bddce6ba1748e64bc30d77f1), [`63796ba`](https://github.com/mastra-ai/mastra/commit/63796ba0fda60253be17535e68f6bbbf1e6ffa09), [`b1ad324`](https://github.com/mastra-ai/mastra/commit/b1ad324d657f3544b0701332aef7eb10e9a36258), [`61c566d`](https://github.com/mastra-ai/mastra/commit/61c566dd2f2cde2b23ed8f139924e530d4202214), [`c24754c`](https://github.com/mastra-ai/mastra/commit/c24754c1fb6fe144e5051e536e98c8a18b0214ac), [`12c61d2`](https://github.com/mastra-ai/mastra/commit/12c61d280c8cb208bc3c8dbcbe5dcc60cf9d1cd0), [`c46eb09`](https://github.com/mastra-ai/mastra/commit/c46eb09ce4987509af57a0ac582c61241a6dd2f1), [`9ee8120`](https://github.com/mastra-ai/mastra/commit/9ee8120ce17f76b9f617489e05a283353742690a), [`d975e92`](https://github.com/mastra-ai/mastra/commit/d975e924d4936f46c386bd3dee39c671720289f6), [`45dd6ee`](https://github.com/mastra-ai/mastra/commit/45dd6ee089bd7df0d0c98a10098e483fd388e04a), [`4e9a228`](https://github.com/mastra-ai/mastra/commit/4e9a2283d5fd6ed1b70a2751eb3dc2cbf82ada20), [`d6ce34a`](https://github.com/mastra-ai/mastra/commit/d6ce34aeceb06ddf3d595a1eed5cc74f481a46a1), [`f95f468`](https://github.com/mastra-ai/mastra/commit/f95f468cf1e7c2b924a13826494f98b8f2ccd581), [`30ed33e`](https://github.com/mastra-ai/mastra/commit/30ed33ee14084a26019aba15fceadda6d6ddefaf), [`04a815f`](https://github.com/mastra-ai/mastra/commit/04a815fc8971d29e97fcdcc5008a1eb472fc00ff), [`1cfa878`](https://github.com/mastra-ai/mastra/commit/1cfa8784d8da0dfaa0317e5048bc48b6084a5ea5), [`9a12ef3`](https://github.com/mastra-ai/mastra/commit/9a12ef3fccf3f4186db0f294f4ee1f02cf4d8db2), [`32d3583`](https://github.com/mastra-ai/mastra/commit/32d358332cb8ac2306b83b73cf3536e74dbd435e), [`7960688`](https://github.com/mastra-ai/mastra/commit/7960688828e04eaf3106e34f7758fa580257eef6), [`91ad69d`](https://github.com/mastra-ai/mastra/commit/91ad69d64994c89199b0c55399e64ed91c61df2f), [`8dc408d`](https://github.com/mastra-ai/mastra/commit/8dc408d34438f9e13297f792c11a5cfd6cf952e1), [`c92def1`](https://github.com/mastra-ai/mastra/commit/c92def10a13c822972c96f0a4ca6ffc1f4258aed), [`63041eb`](https://github.com/mastra-ai/mastra/commit/63041eb4c50b520a0a80e03d4cd6ea99f67715a0), [`c118318`](https://github.com/mastra-ai/mastra/commit/c1183181c9804303db4b511c2e2648f8b714712b), [`c5eaec5`](https://github.com/mastra-ai/mastra/commit/c5eaec5a860d80d0e3805e67db0414b87ac8cbed), [`fc07c64`](https://github.com/mastra-ai/mastra/commit/fc07c6465043e08e99193a6751a01c56ffc2e7a1), [`cced745`](https://github.com/mastra-ai/mastra/commit/cced745a056ec2225c5bc702e32d848847aa8b65), [`542dee2`](https://github.com/mastra-ai/mastra/commit/542dee254167f974ff8cbbbfc0ce10f9a2616a7b), [`3c19dce`](https://github.com/mastra-ai/mastra/commit/3c19dcef8e73062a80627a4927eae3ec11145afd), [`aca2869`](https://github.com/mastra-ai/mastra/commit/aca2869b2031982f3c4a2f52525c9be7cf123ef8), [`a58483c`](https://github.com/mastra-ai/mastra/commit/a58483cff1a9d41fce7c931843f48cb0ac450f64), [`a58483c`](https://github.com/mastra-ai/mastra/commit/a58483cff1a9d41fce7c931843f48cb0ac450f64), [`e6f8450`](https://github.com/mastra-ai/mastra/commit/e6f845074d478527026b18d85031b23353e1d0a4), [`895e9df`](https://github.com/mastra-ai/mastra/commit/895e9dfc17d6f34299eca64e317ded9e5f5e5ef8), [`e66b2ba`](https://github.com/mastra-ai/mastra/commit/e66b2ba100db63eaeab6e21e1ea34b113f2ec781), [`3e8727e`](https://github.com/mastra-ai/mastra/commit/3e8727e11ec1a5d733acedb5c872896394be18c1)]:
+  - @mastra/core@1.62.0
+
+## 0.3.0-alpha.0
+
+### Minor Changes
+
+- Added intent-aware GitHub pull request subscription modes. Review mode follows code revisions, authorized comments, review-thread state, and terminal PR state without CI or mergeability noise, while omitted modes retain working behavior. ([#21542](https://github.com/mastra-ai/mastra/pull/21542))
+
+  ```ts
+  await githubSignals.subscribeThreadToPR({ threadId, resourceId, pr, mode: 'review' });
+  ```
+
+### Patch Changes
+
+- Updated dependencies [[`4ff3ee2`](https://github.com/mastra-ai/mastra/commit/4ff3ee2bff7ed07528b4817f8f49639031c72a4d), [`c24754c`](https://github.com/mastra-ai/mastra/commit/c24754c1fb6fe144e5051e536e98c8a18b0214ac), [`45dd6ee`](https://github.com/mastra-ai/mastra/commit/45dd6ee089bd7df0d0c98a10098e483fd388e04a), [`32d3583`](https://github.com/mastra-ai/mastra/commit/32d358332cb8ac2306b83b73cf3536e74dbd435e), [`aca2869`](https://github.com/mastra-ai/mastra/commit/aca2869b2031982f3c4a2f52525c9be7cf123ef8)]:
+  - @mastra/core@1.62.0-alpha.11
+
 ## 0.2.5
 
 ### Patch Changes

package/dist/index.cjs

@@ -47,8 +47,8 @@ const DEFAULT_AUTHORIZED_PERMISSIONS = [
 ];
 const DEFAULT_AUTHORIZED_BOTS = ["coderabbitai[bot]", "devin-ai-integration[bot]"];
 const PERMISSION_CACHE_TTL_MS = 300 * 1e3;
-/** Notification kinds driven by comment/review activity that should be gated by author permission. */
-const AUTHOR_GATED_NOTIFICATION_KINDS = /* @__PURE__ */ new Set(["pull-request-activity", "pull-request-review-activity"]);
+/** Notification kinds that include comment content and should be gated by author permission. */
+const AUTHOR_GATED_NOTIFICATION_KINDS = /* @__PURE__ */ new Set(["pull-request-activity"]);
 const createGithubTool = _mastra_core_tools.createTool;
 function isPlainObject(value) {
 	return typeof value === "object" && value !== null && !Array.isArray(value);
@@ -60,6 +60,12 @@ function readNumber(value) {
 	if (typeof value === "number" && Number.isInteger(value) && value > 0) return value;
 	if (typeof value === "string" && /^\d+$/.test(value)) return Number(value);
 }
+function normalizeGithubSubscriptionMode(value) {
+	return value === "review" ? "review" : "working";
+}
+function getGithubTerminalState(snapshot) {
+	return snapshot?.state === "closed" || snapshot?.state === "merged" ? snapshot.state : void 0;
+}
 function stableJson(value) {
 	if (Array.isArray(value)) return `[${value.map(stableJson).join(",")}]`;
 	if (isPlainObject(value)) return `{${Object.keys(value).sort().map((key) => `${JSON.stringify(key)}:${stableJson(value[key])}`).join(",")}}`;
@@ -120,6 +126,7 @@ function getGithubMetadata(threadMetadata) {
 			owner,
 			repo,
 			number,
+			mode: normalizeGithubSubscriptionMode(rawSubscription.mode),
 			subscribedAt,
 			updatedAt,
 			lastSubscribeSignalId,
@@ -174,8 +181,11 @@ function getPrLabel(subscription, snapshot) {
 	const pr = `${subscription.owner}/${subscription.repo}#${subscription.number}`;
 	return snapshot?.title ? `${pr}: ${snapshot.title}` : pr;
 }
+function getTerminalNotificationSummary(label, terminalState) {
+	return `${label} was ${terminalState}. This thread has been automatically unsubscribed from this PR. Resubscribe if you still need updates.`;
+}
 function getMergedNotificationSummary(label) {
-	return `${label} was merged. This thread has been automatically unsubscribed from this PR. Resubscribe if you still need updates.`;
+	return getTerminalNotificationSummary(label, "merged");
 }
 /**
 * Removes a hidden block (its delimiters *and* its content) starting at every `open` marker.
@@ -273,7 +283,10 @@ const githubActivityNotificationPriority = {
 	medium: 1
 };
 function getGithubActivityNotificationRank(notification) {
-	return notification.kind === "pull-request-activity" ? 0 : 1;
+	if (notification.kind === "pull-request-activity") return 0;
+	if (notification.kind === "pull-request-code-activity") return 1;
+	if (notification.kind === "pull-request-review-activity") return 2;
+	return 3;
 }
 function compareGithubActivityNotifications(a, b) {
 	if (!a && !b) return 0;
@@ -293,6 +306,28 @@ function classifyGithubCommentActivityNotification(input) {
 		summary
 	};
 }
+function classifyGithubTerminalActivityNotification(input) {
+	return {
+		kind: input.terminalState === "merged" ? "pull-request-merged" : "pull-request-closed",
+		priority: "high",
+		summary: getTerminalNotificationSummary(getPrLabel(input.subscription, input.snapshot), input.terminalState)
+	};
+}
+function classifyGithubHeadActivityNotification(input) {
+	return {
+		kind: "pull-request-code-activity",
+		priority: "medium",
+		summary: `New head revision for ${getPrLabel(input.subscription, input.snapshot)}.`
+	};
+}
+function classifyGithubReviewStateActivityNotification(input) {
+	const count = input.snapshot.unresolvedReviewThreads ?? 0;
+	return {
+		kind: "pull-request-review-activity",
+		priority: "medium",
+		summary: count === 0 ? `All review threads are resolved for ${getPrLabel(input.subscription, input.snapshot)}.` : `${getPrLabel(input.subscription, input.snapshot)} has ${count} unresolved review ${count === 1 ? "thread" : "threads"}.`
+	};
+}
 function getCheckUpdatedTime(check) {
 	const value = check.updatedAt ? Date.parse(check.updatedAt) : NaN;
 	return Number.isFinite(value) ? value : 0;
@@ -703,6 +738,7 @@ var GithubSignals = class extends _mastra_core_signals.SignalProvider {
 	static signals = {
 		subscribeToPR(input) {
 			const normalized = typeof input === "number" ? { number: input } : input;
+			const mode = normalizeGithubSubscriptionMode(normalized.mode);
 			return {
 				type: "reactive",
 				tagName: GITHUB_SUBSCRIBE_PR_TAG,
@@ -710,11 +746,13 @@ var GithubSignals = class extends _mastra_core_signals.SignalProvider {
 				attributes: {
 					...normalized.owner ? { owner: normalized.owner } : {},
 					...normalized.repo ? { repo: normalized.repo } : {},
-					number: normalized.number
+					number: normalized.number,
+					mode
 				},
 				metadata: { github: {
 					action: "subscribeToPR",
-					...normalized
+					...normalized,
+					mode
 				} }
 			};
 		},
@@ -794,6 +832,7 @@ var GithubSignals = class extends _mastra_core_signals.SignalProvider {
 		return this.#subscribe({
 			id: `github-command-subscribe-${(0, crypto.randomUUID)()}`,
 			...pr,
+			mode: normalizeGithubSubscriptionMode(input.mode),
 			threadId: input.threadId,
 			resourceId: input.resourceId
 		});
@@ -898,10 +937,11 @@ var GithubSignals = class extends _mastra_core_signals.SignalProvider {
 			abortSignal: args.abortSignal
 		});
 		if (result.alreadyProcessed) return { tools };
+		const terminalMessage = result.terminalState ? `Not subscribed to ${result.owner}/${result.repo}#${result.number} in review mode because it is already ${result.terminalState}.` : void 0;
 		await this.#sendStatus(args, result, {
-			status: result.syncResult?.ok === false ? "sync_error" : "subscribed",
+			status: result.terminalState ? "not_subscribed_terminal" : result.syncResult?.ok === false ? "sync_error" : "subscribed",
 			action: "subscribeToPR",
-			message: result.syncResult?.ok === false ? `Subscribed to ${result.owner}/${result.repo}#${result.number}, but gitcrawl sync failed: ${result.syncResult.error}` : `Subscribed to ${result.owner}/${result.repo}#${result.number}.`
+			message: terminalMessage ?? (result.syncResult?.ok === false ? `Subscribed to ${result.owner}/${result.repo}#${result.number} in ${result.mode} mode, but gitcrawl sync failed: ${result.syncResult.error}` : `Subscribed to ${result.owner}/${result.repo}#${result.number} in ${result.mode} mode.`)
 		});
 		return { tools };
 	}
@@ -941,8 +981,12 @@ var GithubSignals = class extends _mastra_core_signals.SignalProvider {
 		await args.sendSignal?.({
 			type: "reactive",
 			tagName: "system-reminder",
-			contents: `Looks like you're working with ${repository.owner}/${repository.repo}#${evidence.number}. Use /github subscribe ${evidence.number} or the github_subscribe_pr tool to follow updates.`,
-			attributes: { type: "github-subscription-hint" },
+			contents: `Choose whether to follow ${repository.owner}/${repository.repo}#${evidence.number}: use /github subscribe ${evidence.number} --mode review for new commits, authorized latest PR comments, review-thread-state changes including all threads resolved, and PR close or merge updates; use --mode working for all actionable PR activity. Do not subscribe for a one-off inspection.`,
+			attributes: {
+				type: "github-subscription-hint",
+				availableModes: "review,working",
+				defaultMode: null
+			},
 			metadata: { github: {
 				action: "subscriptionHint",
 				owner: repository.owner,
@@ -973,11 +1017,12 @@ var GithubSignals = class extends _mastra_core_signals.SignalProvider {
 			...args.tools,
 			github_subscribe_pr: createGithubTool({
 				id: "github_subscribe_pr",
-				description: "Subscribe this thread to a GitHub pull request. Syncs only the requested PR with gitcrawl and stores the subscription on the thread.",
+				description: "Subscribe this thread to a GitHub pull request. Use review mode for new commits, authorized latest PR comments, review-thread-state changes including all threads resolved, and PR close or merge updates. Use working mode for all actionable PR activity. Omitted mode defaults to working. Do not subscribe for a one-off inspection.",
 				inputSchema: zod.default.object({
 					number: zod.default.number().int().positive(),
 					owner: zod.default.string().optional(),
-					repo: zod.default.string().optional()
+					repo: zod.default.string().optional(),
+					mode: zod.default.enum(["working", "review"]).optional()
 				}),
 				execute: async (input, context) => {
 					const executionThreadContext = getExecutionThreadContext(context);
@@ -986,16 +1031,28 @@ var GithubSignals = class extends _mastra_core_signals.SignalProvider {
 						owner: input.owner,
 						repo: input.repo,
 						number: input.number,
+						mode: normalizeGithubSubscriptionMode(input.mode),
 						threadId: executionThreadContext.threadId,
 						resourceId: executionThreadContext.resourceId
 					});
+					if (result.terminalState) return {
+						subscribed: false,
+						mode: result.mode,
+						terminalState: result.terminalState,
+						reason: "terminal",
+						owner: result.owner,
+						repo: result.repo,
+						number: result.number,
+						message: `Not subscribed to ${result.owner}/${result.repo}#${result.number} in review mode because it is already ${result.terminalState}.`
+					};
 					return {
 						subscribed: true,
+						mode: result.mode,
 						owner: result.owner,
 						repo: result.repo,
 						number: result.number,
 						syncStatus: result.syncResult?.ok === false ? "error" : result.syncResult ? "success" : void 0,
-						message: result.syncResult?.ok === false ? `Subscribed to ${result.owner}/${result.repo}#${result.number}, but gitcrawl sync failed: ${result.syncResult.error}` : `Subscribed to ${result.owner}/${result.repo}#${result.number}.`
+						message: result.syncResult?.ok === false ? `Subscribed to ${result.owner}/${result.repo}#${result.number} in ${result.mode} mode, but gitcrawl sync failed: ${result.syncResult.error}` : `Subscribed to ${result.owner}/${result.repo}#${result.number} in ${result.mode} mode.`
 					};
 				}
 			}),
@@ -1135,15 +1192,19 @@ var GithubSignals = class extends _mastra_core_signals.SignalProvider {
 				const previousContentHash = subscription.lastObservedContentHash;
 				const previousThreadContentHash = subscription.lastObservedThreadContentHash;
 				const previousHeadSha = subscription.lastObservedHeadSha;
+				const previousReviewStateHash = subscription.lastObservedReviewStateHash;
 				const latestCommentTimestampChanged = !!previousGithubUpdatedAt && !!snapshot?.latestCommentUpdatedAt && Date.parse(snapshot.latestCommentUpdatedAt) > Date.parse(previousGithubUpdatedAt);
 				const existingBotCommentEdited = latestCommentTimestampChanged && !!snapshot && isExistingBotCommentEdit(subscription, snapshot);
 				const latestCommentChanged = latestCommentTimestampChanged && !existingBotCommentEdited;
+				const headChanged = !!previousHeadSha && !!snapshot?.headSha && previousHeadSha !== snapshot.headSha;
+				const reviewStateChanged = !!previousReviewStateHash && !!snapshot?.reviewStateHash && previousReviewStateHash !== snapshot.reviewStateHash;
 				if (snapshot) applySnapshotCursor(nextSubscription, snapshot);
 				const isFirstObservation = syncResult.ok && snapshot && !previousGithubUpdatedAt && !previousContentHash;
 				const legacyAggregateChanged = previousContentHash && snapshot?.contentHash && previousContentHash !== snapshot.contentHash && !previousThreadContentHash && !previousHeadSha;
-				const changed = isFirstObservation || syncResult.ok && snapshot && (legacyAggregateChanged || latestCommentChanged || previousThreadContentHash && snapshot.threadContentHash && previousThreadContentHash !== snapshot.threadContentHash && !existingBotCommentEdited || previousHeadSha && snapshot.headSha && previousHeadSha !== snapshot.headSha || subscription.lastObservedState && snapshot.state && subscription.lastObservedState !== snapshot.state || hasMeaningfulMergeableStateChange(subscription.lastObservedMergeableState, snapshot.mergeableState) || subscription.lastObservedCiState && snapshot.ciState && subscription.lastObservedCiState !== snapshot.ciState || subscription.lastObservedReviewStateHash && snapshot.reviewStateHash && subscription.lastObservedReviewStateHash !== snapshot.reviewStateHash);
-				let shouldKeepSubscription = true;
-				if (changed && snapshot && isCurrentGeneration()) {
+				const changed = isFirstObservation || syncResult.ok && snapshot && (legacyAggregateChanged || latestCommentChanged || previousThreadContentHash && snapshot.threadContentHash && previousThreadContentHash !== snapshot.threadContentHash && !existingBotCommentEdited || headChanged || subscription.lastObservedState && snapshot.state && subscription.lastObservedState !== snapshot.state || hasMeaningfulMergeableStateChange(subscription.lastObservedMergeableState, snapshot.mergeableState) || subscription.lastObservedCiState && snapshot.ciState && subscription.lastObservedCiState !== snapshot.ciState || reviewStateChanged);
+				const terminalState = subscription.mode === "review" ? getGithubTerminalState(snapshot) : void 0;
+				let shouldKeepSubscription = !terminalState;
+				if ((changed || terminalState) && snapshot && !(subscription.mode === "review" && isFirstObservation && !terminalState) && isCurrentGeneration()) {
 					const notifications = await this.#sendActivityNotifications({
 						polling: input,
 						subscription,
@@ -1151,6 +1212,9 @@ var GithubSignals = class extends _mastra_core_signals.SignalProvider {
 						previousGithubUpdatedAt,
 						previousContentHash,
 						latestCommentChanged,
+						headChanged,
+						reviewStateChanged,
+						terminalState,
 						isCurrentGeneration
 					});
 					if (!isCurrentGeneration()) return 0;
@@ -1160,7 +1224,7 @@ var GithubSignals = class extends _mastra_core_signals.SignalProvider {
 						nextSubscription.lastNotificationKind = primaryNotification.kind;
 						nextSubscription.lastNotificationPriority = primaryNotification.priority;
 						nextSubscription.lastNotificationSummary = primaryNotification.summary;
-						shouldKeepSubscription = notifications.every((notification) => notification.kind !== "pull-request-merged");
+						if (!terminalState) shouldKeepSubscription = notifications.every((notification) => notification.kind !== "pull-request-merged");
 					}
 				}
 				if (shouldKeepSubscription) subscriptions.push(nextSubscription);
@@ -1210,6 +1274,7 @@ var GithubSignals = class extends _mastra_core_signals.SignalProvider {
 				owner: input.subscription.owner,
 				repo: input.subscription.repo,
 				number: input.subscription.number,
+				mode: input.subscription.mode,
 				...input.snapshot.title ? { title: input.snapshot.title } : {},
 				...input.snapshot.state ? { state: input.snapshot.state } : {},
 				...input.snapshot.htmlUrl ? { url: input.snapshot.htmlUrl } : {},
@@ -1229,6 +1294,7 @@ var GithubSignals = class extends _mastra_core_signals.SignalProvider {
 				owner: input.subscription.owner,
 				repo: input.subscription.repo,
 				number: input.subscription.number,
+				mode: input.subscription.mode,
 				title: input.snapshot.title,
 				state: input.snapshot.state,
 				htmlUrl: input.snapshot.htmlUrl,
@@ -1371,14 +1437,36 @@ var GithubSignals = class extends _mastra_core_signals.SignalProvider {
 	async #sendActivityNotifications(input) {
 		const agent = this.#getNotificationAgent(input.polling);
 		if (!agent?.sendNotificationSignal) return [];
-		const notifications = [classifyGithubActivityNotification({
-			subscription: input.subscription,
-			snapshot: input.snapshot
-		})];
-		if (input.latestCommentChanged && notifications[0]?.kind !== "pull-request-activity") notifications.push(classifyGithubCommentActivityNotification({
+		const notifications = [];
+		if (input.subscription.mode === "review") if (input.terminalState) notifications.push(classifyGithubTerminalActivityNotification({
 			subscription: input.subscription,
-			snapshot: input.snapshot
+			snapshot: input.snapshot,
+			terminalState: input.terminalState
 		}));
+		else {
+			if (input.latestCommentChanged) notifications.push(classifyGithubCommentActivityNotification({
+				subscription: input.subscription,
+				snapshot: input.snapshot
+			}));
+			if (input.headChanged) notifications.push(classifyGithubHeadActivityNotification({
+				subscription: input.subscription,
+				snapshot: input.snapshot
+			}));
+			if (input.reviewStateChanged) notifications.push(classifyGithubReviewStateActivityNotification({
+				subscription: input.subscription,
+				snapshot: input.snapshot
+			}));
+		}
+		else {
+			notifications.push(classifyGithubActivityNotification({
+				subscription: input.subscription,
+				snapshot: input.snapshot
+			}));
+			if (input.latestCommentChanged && notifications[0]?.kind !== "pull-request-activity") notifications.push(classifyGithubCommentActivityNotification({
+				subscription: input.subscription,
+				snapshot: input.snapshot
+			}));
+		}
 		const sent = [];
 		const notificationInputs = [];
 		for (const notification of notifications.sort(compareGithubActivityNotifications)) {
@@ -1418,6 +1506,7 @@ var GithubSignals = class extends _mastra_core_signals.SignalProvider {
 		return sent;
 	}
 	async #subscribe(input) {
+		const mode = normalizeGithubSubscriptionMode(input.mode);
 		const { owner, repo } = await this.#resolveRepository(input);
 		const { threadStore, loadedThread } = await this.#loadThread(input);
 		const githubMetadata = getGithubMetadata(loadedThread.metadata);
@@ -1427,6 +1516,7 @@ var GithubSignals = class extends _mastra_core_signals.SignalProvider {
 			owner,
 			repo,
 			number: input.number,
+			mode: existing.mode,
 			subscription: existing,
 			alreadyProcessed: true
 		};
@@ -1435,6 +1525,7 @@ var GithubSignals = class extends _mastra_core_signals.SignalProvider {
 			owner,
 			repo,
 			number: input.number,
+			mode,
 			subscribedAt: existing?.subscribedAt ?? now,
 			updatedAt: now,
 			lastSubscribeSignalId: input.id,
@@ -1478,6 +1569,35 @@ var GithubSignals = class extends _mastra_core_signals.SignalProvider {
 			baselineSnapshot = snapshot;
 			if (snapshot) applySnapshotCursor(subscription, snapshot);
 		} else subscription.lastSyncStatus = "skipped";
+		const terminalState = mode === "review" ? getGithubTerminalState(baselineSnapshot) : void 0;
+		if (terminalState) {
+			const subscriptions = githubMetadata.subscriptions.filter((candidate) => !(candidate.owner === owner && candidate.repo === repo && candidate.number === input.number));
+			await threadStore.saveThread({ thread: {
+				...loadedThread,
+				id: input.threadId,
+				resourceId: input.resourceId,
+				createdAt: loadedThread.createdAt ?? /* @__PURE__ */ new Date(),
+				updatedAt: /* @__PURE__ */ new Date(),
+				metadata: setGithubMetadata(loadedThread.metadata, { subscriptions })
+			} });
+			this.#notifySubscriptionsChanged({
+				threadId: input.threadId,
+				resourceId: input.resourceId,
+				subscriptions
+			});
+			if (subscriptions.length === 0) this.stopPollingForThread({
+				threadId: input.threadId,
+				resourceId: input.resourceId
+			});
+			return {
+				owner,
+				repo,
+				number: input.number,
+				mode,
+				terminalState,
+				syncResult
+			};
+		}
 		const subscriptions = [subscription];
 		await threadStore.saveThread({ thread: {
 			...loadedThread,
@@ -1492,7 +1612,7 @@ var GithubSignals = class extends _mastra_core_signals.SignalProvider {
 			resourceId: input.resourceId,
 			subscriptions
 		});
-		if (baselineSnapshot) await this.#sendBaselineNotification({
+		if (baselineSnapshot && mode === "working") await this.#sendBaselineNotification({
 			threadId: input.threadId,
 			resourceId: input.resourceId,
 			subscription,
@@ -1506,6 +1626,7 @@ var GithubSignals = class extends _mastra_core_signals.SignalProvider {
 			owner,
 			repo,
 			number: input.number,
+			mode,
 			subscription,
 			syncResult
 		};
@@ -1558,7 +1679,8 @@ var GithubSignals = class extends _mastra_core_signals.SignalProvider {
 			id: readString(signal.id) ?? message.id,
 			owner: readString(attributes.owner) ?? readString(github.owner),
 			repo: readString(attributes.repo) ?? readString(github.repo),
-			number
+			number,
+			...signal.tagName === "github-subscribe-pr" ? { mode: normalizeGithubSubscriptionMode(attributes.mode ?? github.mode) } : {}
 		};
 	}
 	async #sendStatus(args, signal, status) {
@@ -1570,14 +1692,18 @@ var GithubSignals = class extends _mastra_core_signals.SignalProvider {
 				status: status.status,
 				owner: signal.owner,
 				repo: signal.repo,
-				number: signal.number
+				number: signal.number,
+				...signal.mode ? { mode: signal.mode } : {},
+				...signal.terminalState ? { terminalState: signal.terminalState } : {}
 			},
 			metadata: { github: {
 				action: status.action,
 				status: status.status,
 				owner: signal.owner,
 				repo: signal.repo,
-				number: signal.number
+				number: signal.number,
+				...signal.mode ? { mode: signal.mode } : {},
+				...signal.terminalState ? { terminalState: signal.terminalState } : {}
 			} }
 		});
 	}