Merge pull request #78 from Boring-Software-Inc/chore/remove-ascii-comments

chore: replace all ascii comments with plain

Author: ripgrim

apps/web/autumn.config.ts

@@ -1,6 +1,6 @@
 import { feature, item, plan } from "atmn"
 
-// ─── Features ───────────────────────────────────────────────
+// Features
 
 // Tracks AI spend in cents. With 1.25x markup on gpt-4o-mini:
 
@@ -19,7 +19,7 @@ export const custom_rules = feature({
   consumable: false,
 })
 
-// ─── Plans ──────────────────────────────────────────────────
+// Plans
 
 // Free: 100¢ = $1.00 of AI spend/month
 

apps/web/src/integrations/trpc/routers/workflows.ts

@@ -717,7 +717,7 @@ export const workflowsRouter = {
       }
     }),
 
-  // ─── Admin endpoints (internal data collection) ────────────
+  // Admin endpoints (internal data collection)
 
   /** Scan an entire repo's contributors — admin only, no repo ownership check */
   adminScanRepo: adminProcedure

packages/core/src/contributor-score.ts

@@ -42,15 +42,15 @@ export interface ScoreInput {
   allowedCount: number
   nearMissCount: number
 
-  // ─── Change 1: PR substance (split volume from quality) ─────
+  // Change 1: PR substance (split volume from quality)
   /** Summary of merged PRs with quality weighting. Null = use flat mergedPrCount. */
   mergedPrSummary?: {
     total: number
     /** Each PR's multiplier summed based on target repo quality tier */
     qualityWeightedCount: number
   } | null
 
-  // ─── Change 2: Spray detection temporal data ────────────────
+  // Change 2: Spray detection temporal data
   /** Temporal PR data for spray pattern detection. Null = skip spray checks. */
   prTemporalData?: {
     /** Intervals in seconds between consecutive PR creation timestamps */
@@ -65,7 +65,7 @@ export interface ScoreInput {
     reposInDensestWindow: number
   } | null
 
-  // ─── Change 3: Repo history with timestamps for decay ───────
+  // Change 3: Repo history with timestamps for decay
   /** Timestamped events for time-decay scoring. Null = fall back to flat counts. */
   repoEvents?: Array<{
     type: "allowed" | "blocked" | "near-miss" | "cleared"
@@ -546,7 +546,7 @@ function scoreRedFlags(input: ScoreInput, sink: ScoreLineItem[]): number {
     )
   }
 
-  // ─── Spray detection (requires prTemporalData) ──────────────
+  // Spray detection (requires prTemporalData)
   if (input.prTemporalData) {
     const td = input.prTemporalData
 

packages/core/src/filter-pipeline.ts

@@ -40,7 +40,7 @@ import { loadPrefsForInstallation } from "./pr-comment-loader"
 
 const APP_BASE_URL = env.BETTER_AUTH_URL ?? ""
 
-// ─── Scope helper ──────────────────────────────────────────────
+// Scope helper
 
 type ScopeKey = "pullRequests" | "issues" | "comments"
 
@@ -81,7 +81,7 @@ function ruleApplies(
   return scope[key]
 }
 
-// ─── Types ─────────────────────────────────────────────────────
+// Types
 
 export interface WebhookContext {
   installationId: number
@@ -189,7 +189,7 @@ function resolveOutcome(
   return "logged"
 }
 
-// ─── Near-miss threshold ───────────────────────────────────────
+// Near-miss threshold
 // A user is "near miss" if their value is within 20% of triggering.
 const NEAR_MISS_RATIO = 0.2
 
@@ -207,7 +207,7 @@ function isNearMissMax(actual: number, limit: number): boolean {
   return actual >= limit * (1 - NEAR_MISS_RATIO)
 }
 
-// ─── Content analysis helpers ──────────────────────────────────
+// Content analysis helpers
 
 /**
  * Detect the dominant script/language of text using Unicode code-point ranges.
@@ -355,7 +355,7 @@ function isLikelyLanguage(text: string, language: string): boolean {
   return detection.dominant === expected && detection.confidence > 0.3
 }
 
-// ─── Crypto address detection ──────────────────────────────────
+// Crypto address detection
 
 const CRYPTO_PATTERNS: { name: string; pattern: RegExp }[] = [
   // Bitcoin (legacy P2PKH/P2SH + SegWit bech32)
@@ -386,7 +386,7 @@ function detectCryptoAddress(
   return null
 }
 
-// ─── Pipeline ──────────────────────────────────────────────────
+// Pipeline
 
 /**
  * Run all enabled rules against a GitHub user, collecting detailed
@@ -569,7 +569,7 @@ export async function runFilterPipeline(
     contentScope: scope,
   }
 
-  // ─── autoWhitelistGlobalVouches ───────────────────────────
+  // autoWhitelistGlobalVouches
   if (config.autoWhitelistGlobalVouches.enabled) {
     const minVouches = config.autoWhitelistGlobalVouches.minVouches
     const vouchRows = await db
@@ -597,7 +597,7 @@ export async function runFilterPipeline(
     }
   }
 
-  // ─── vouchedUsersOnly ──────────────────────────────────────
+  // vouchedUsersOnly
   // Non-vouched users are rejected before any per-user GitHub lookups.
   // Whitelisted users already returned above (covers "repo" scope).
   // When vouchScope is "global" or "both", also check global vouches.
@@ -765,7 +765,7 @@ export async function runFilterPipeline(
     }
   }
 
-  // ─── accountAge ────────────────────────────────────────────
+  // accountAge
   if (ruleApplies(config.accountAge, contentType, scope) && ghUser) {
     rulesChecked++
     const createdAt = new Date(ghUser.created_at as string)
@@ -797,7 +797,7 @@ export async function runFilterPipeline(
     }
   }
 
-  // ─── minMergedPrs ──────────────────────────────────────────
+  // minMergedPrs
   if (ruleApplies(config.minMergedPrs, contentType, scope)) {
     rulesChecked++
     try {
@@ -830,7 +830,7 @@ export async function runFilterPipeline(
     }
   }
 
-  // ─── languageRequirement ───────────────────────────────────
+  // languageRequirement
   if (
     ruleApplies(config.languageRequirement, contentType, scope) &&
     contentText &&
@@ -860,7 +860,7 @@ export async function runFilterPipeline(
     }
   }
 
-  // ─── maxPrsPerDay ──────────────────────────────────────────
+  // maxPrsPerDay
   if (ruleApplies(config.maxPrsPerDay, contentType, scope)) {
     rulesChecked++
     try {
@@ -897,7 +897,7 @@ export async function runFilterPipeline(
     }
   }
 
-  // ─── maxFilesChanged ───────────────────────────────────────
+  // maxFilesChanged
   if (ruleApplies(config.maxFilesChanged, contentType, scope) && ctx.prNumber) {
     rulesChecked++
     try {
@@ -940,7 +940,7 @@ export async function runFilterPipeline(
     }
   }
 
-  // ─── repoActivityMinimum ───────────────────────────────────
+  // repoActivityMinimum
   if (ruleApplies(config.repoActivityMinimum, contentType, scope)) {
     rulesChecked++
     try {
@@ -977,7 +977,7 @@ export async function runFilterPipeline(
     }
   }
 
-  // ─── requireProfileReadme ──────────────────────────────────
+  // requireProfileReadme
   if (ruleApplies(config.requireProfileReadme, contentType, scope)) {
     rulesChecked++
     try {
@@ -1010,7 +1010,7 @@ export async function runFilterPipeline(
     }
   }
 
-  // ─── cryptoAddressDetection ────────────────────────────────
+  // cryptoAddressDetection
   if (
     ruleApplies(config.cryptoAddressDetection, contentType, scope) &&
     contentText
@@ -1124,7 +1124,7 @@ export async function runFilterPipeline(
   }
 }
 
-// ─── Pipeline event logging ────────────────────────────────────
+// Pipeline event logging
 
 /**
  * Generate a unique pipeline ID for grouping events from the same evaluation.
@@ -1292,7 +1292,7 @@ async function logPipelineEvents(
   await logEvents(eventBatch)
 }
 
-// ─── Webhook action handlers ───────────────────────────────────
+// Webhook action handlers
 
 /**
  * Execute the resolved action on a PR/issue/comment based on the pipeline result.

packages/core/src/rules.test.ts

@@ -16,7 +16,7 @@ import {
   cleanForLanguageDetection,
 } from "./language-detection"
 
-// ─── Crypto Patterns (exact copy from filter-pipeline.ts) ──────
+// Crypto Patterns (exact copy from filter-pipeline.ts)
 
 const CRYPTO_PATTERNS: { name: string; pattern: RegExp }[] = [
   { name: "Bitcoin", pattern: /\b(bc1|[13])[a-zA-HJ-NP-Z0-9]{25,39}\b/ },
@@ -36,7 +36,7 @@ function detectCryptoAddress(
   return null
 }
 
-// ─── Near-miss helpers (exact copy from filter-pipeline.ts) ────
+// Near-miss helpers (exact copy from filter-pipeline.ts)
 
 const NEAR_MISS_RATIO = 0.2
 

packages/db/src/schema/auth.ts

@@ -1,7 +1,7 @@
 import { boolean, pgTable, text, timestamp } from "drizzle-orm/pg-core"
 import { relations } from "drizzle-orm"
 
-// ─── Better Auth core tables ─────────────────────────────────────
+// Better Auth core tables
 // Managed by better-auth. We define them here so Drizzle is aware of
 // them for relations / migrations.
 

packages/db/src/schema/oauth.ts

@@ -1,7 +1,7 @@
 import { boolean, pgTable, text, timestamp } from "drizzle-orm/pg-core"
 import { user } from "./auth"
 
-// ─── Better Auth MCP plugin (OIDC provider schema) ───────────────
+// Better Auth MCP plugin (OIDC provider schema)
 
 export const oauthApplication = pgTable("oauth_application", {
   id: text("id").primaryKey(),

packages/db/src/schema/orgs.ts

@@ -2,7 +2,7 @@ import { pgTable, text, timestamp } from "drizzle-orm/pg-core"
 import { relations } from "drizzle-orm"
 import { user } from "./auth"
 
-// ─── Better Auth organization plugin tables ──────────────────────
+// Better Auth organization plugin tables
 
 export const organization = pgTable("organization", {
   id: text("id").primaryKey(),

packages/env/src/server.ts

@@ -5,7 +5,7 @@ import { createEnv } from "@t3-oss/env-core"
 import { config as loadDotenv } from "dotenv"
 import { z } from "zod"
 
-// ─── Root .env autoload ──────────────────────────────────────────
+// Root .env autoload
 // Walk up from this file to the monorepo root (marked by pnpm-workspace.yaml)
 // and load .env from there. dotenv is idempotent — vars already in
 // process.env (e.g. injected by Vite / Nitro / CI) win.