fix: address codex review round 2

- Fix bugcop sflow type narrowing: use undefined + non-null
  assertion to avoid tsc error with sflow's filter
- Fix getActivatedShell: use /bin/sh -c wrapper so source
  and shell syntax work correctly (Bun $ escapes interpolated
  values as single arguments)
- Revert contributor API sort+limit: collection is small and
  admin-only, sorting by updatedAt misses errorAt-only updates

Amp-Thread-ID: https://ampcode.com/threads/T-019da183-14cd-7179-bb7c-9178dc8494d1
Co-authored-by: Amp <amp@ampcode.com>

Author: snomiao

app/api/router.ts

@@ -87,7 +87,7 @@ export const router = t.router({
     .query(async () => {
       const { GithubContributorAnalyzeTask } =
         await import("../tasks/github-contributor-analyze/GithubContributorAnalyzeTask");
-      return await GithubContributorAnalyzeTask.find({}).sort({ updatedAt: -1 }).limit(500).toArray();
+      return await GithubContributorAnalyzeTask.find({}).toArray();
     }),
 
   githubContributorAnalyze: t.procedure

run/gh-bugcop/gh-bugcop.tsx

@@ -413,13 +413,13 @@ export default async function runGithubBugcopTask() {
       } catch (e) {
         if ((e as { status?: number }).status === 404) {
           tlog(chalk.yellow(`Issue not found (deleted/transferred?): ${issueUrl}`));
-          return null;
+          return undefined;
         }
         throw e;
       }
     })
-    .filter((issue): issue is NonNullable<typeof issue> => issue !== null)
-    .forEach(processIssue)
+    .filter((issue) => issue !== undefined)
+    .forEach((issue) => processIssue(issue!))
     .toArray();
 
   tlog(

src/getActivatedShell.ts

@@ -3,14 +3,15 @@ import { getActivateCMD } from "./cli/getActivateCMD";
 
 if (import.meta.main) {
   const activate = getActivateCMD();
-  const p = await $`${activate} && comfy-cli --version`;
+  // Use .nothrow() + raw shell string to allow `source` on POSIX
+  const p = await $`/bin/sh -c ${`${activate} && comfy-cli --version`}`;
   console.log(p.stdout.toString());
 }
 
 export function getActivatedShell() {
   const activate = getActivateCMD();
   return (strings: TemplateStringsArray, ...values: unknown[]) => {
     const cmd = strings.reduce((acc, str, i) => acc + str + (values[i] ?? ""), "");
-    return $`${activate} && ${cmd}`;
+    return $`/bin/sh -c ${`${activate} && ${cmd}`}`;
   };
 }