cyrusagents
diff --git a/‎CHANGELOG.internal.md‎
Lines changed: 20 additions & 0 deletions b/‎CHANGELOG.internal.md‎
Lines changed: 20 additions & 0 deletions
diff --git a/‎CHANGELOG.md‎
Lines changed: 80 additions & 0 deletions b/‎CHANGELOG.md‎
Lines changed: 80 additions & 0 deletions
diff --git a/‎apps/cli/package.json‎
Lines changed: 1 addition & 1 deletion b/‎apps/cli/package.json‎
Lines changed: 1 addition & 1 deletion
diff --git a/‎apps/cli/src/services/Logger.ts‎
Lines changed: 40 additions & 84 deletions b/‎apps/cli/src/services/Logger.ts‎
Lines changed: 40 additions & 84 deletions
diff --git a/‎package-lock.json‎
Lines changed: 4 additions & 4 deletions b/‎package-lock.json‎
Lines changed: 4 additions & 4 deletions
diff --git a/‎packages/claude-runner/package.json‎
Lines changed: 3 additions & 3 deletions b/‎packages/claude-runner/package.json‎
Lines changed: 3 additions & 3 deletions
@@ -5,14 +5,34 @@ This changelog documents internal development changes, refactors, tooling update
 ## [Unreleased]
 
 ### Changed
+- Refactored logging across all packages to use a dedicated `ILogger` interface and `Logger` implementation in `packages/core/src/logging/`. Replaced direct `console.log`/`console.error` calls in EdgeWorker, AgentSessionManager, ClaudeRunner, GitService, RepositoryRouter, SharedApplicationServer, SharedWebhookServer, WorktreeIncludeService, ProcedureAnalyzer, AskUserQuestionHandler, LinearEventTransport, and LinearIssueTrackerService with structured logger calls. Log level is configurable via the `CYRUS_LOG_LEVEL` environment variable (DEBUG, INFO, WARN, ERROR, SILENT).
+- Added source context (session ID, platform, issue identifier, repository) to log messages via `logger.withContext()`, enabling easier debugging and log filtering across concurrent sessions
 - Updated `CyrusAgentSession` schema to v3.0: renamed `linearAgentActivitySessionId` to `id`, added optional `externalSessionId` for tracker-specific IDs, added optional `issueContext` object for issue metadata, made `issue` and `issueId` optional to support standalone sessions ([CYPACK-728](https://linear.app/ceedar/issue/CYPACK-728), [#770](https://github.com/ceedaragents/cyrus/pull/770))
 - Updated `PersistenceManager` to v3.0 format with automatic migration from v2.0, preserving all existing session data during migration ([CYPACK-728](https://linear.app/ceedar/issue/CYPACK-728), [#770](https://github.com/ceedaragents/cyrus/pull/770))
+- GitHub webhook handling now uses forwarded installation tokens: `GitHubEventTransport` extracts `X-GitHub-Installation-Token` header from CYHOST webhooks and includes it in emitted events, `EdgeWorker.postGitHubReply()` and `EdgeWorker.fetchPRBranchRef()` prefer the forwarded token over `process.env.GITHUB_TOKEN`, enabling self-hosted Cyrus instances to post PR comment replies and fetch PR branch details using short-lived (1-hour) GitHub App installation tokens ([CYPACK-773](https://linear.app/ceedar/issue/CYPACK-773), [#821](https://github.com/ceedaragents/cyrus/pull/821), [CYPACK-774](https://linear.app/ceedar/issue/CYPACK-774), [#822](https://github.com/ceedaragents/cyrus/pull/822))
 
 ### Added
+- New `cyrus-github-event-transport` package: EventEmitter-based transport for receiving and verifying forwarded GitHub webhooks, with proxy (Bearer token) and signature (HMAC-SHA256) verification modes, a `GitHubCommentService` for posting replies via GitHub REST API, and utility functions for extracting webhook payload data. ([CYPACK-772](https://linear.app/ceedar/issue/CYPACK-772), [#820](https://github.com/ceedaragents/cyrus/pull/820))
+- EdgeWorker GitHub webhook integration: `/github-webhook` endpoint, session creation flow for PR comments, git worktree checkout for PR branches, and reply posting via GitHub API. ([CYPACK-772](https://linear.app/ceedar/issue/CYPACK-772), [#820](https://github.com/ceedaragents/cyrus/pull/820))
+- Subroutine result text is now stored in procedure history when advancing between subroutines. On error results (e.g. `error_max_turns` from single-turn subroutines), `AgentSessionManager` recovers by using the last completed subroutine's result via `ProcedureAnalyzer.getLastSubroutineResult()`, allowing the procedure to continue to completion instead of failing
 - Created `GlobalSessionRegistry` class for centralized session storage across all repositories, enabling cross-repository session lookups in orchestrator workflows ([CYPACK-725](https://linear.app/ceedar/issue/CYPACK-725), [#766](https://github.com/ceedaragents/cyrus/pull/766))
 - Extracted `IActivitySink` interface and `LinearActivitySink` implementation to decouple activity posting from `IIssueTrackerService`, enabling multiple activity sinks to receive session activities ([CYPACK-726](https://linear.app/ceedar/issue/CYPACK-726), [#767](https://github.com/ceedaragents/cyrus/pull/767))
 - Integrated `GlobalSessionRegistry` with `EdgeWorker`, making it the single source of truth for parent-child session mappings and cross-repository session lookups ([CYPACK-727](https://linear.app/ceedar/issue/CYPACK-727), [#769](https://github.com/ceedaragents/cyrus/pull/769))
 
+## [0.2.21] - 2026-02-09
+
+### Changed
+- Refactored formatting strategy from TodoWrite to Task tools (TaskCreate, TaskUpdate, TaskList, TaskGet). Added `formatTaskParameter()` method to IMessageFormatter interface and updated AgentSessionManager to handle Task tools as thought activities. ([CYPACK-788](https://linear.app/ceedar/issue/CYPACK-788), [#837](https://github.com/ceedaragents/cyrus/pull/837))
+- Redesigned TaskCreate formatting for parallel execution (concise `⏳ **subject**` checklist items), improved TaskUpdate/TaskGet to show subject names with status emojis, added ToolSearch formatting (`🔍 Loading`/`🔍 Searching tools`) rendered as non-ephemeral thought in AgentSessionManager, and added TaskOutput formatting (`📤 Waiting for`/`📤 Checking`). Updated both ClaudeMessageFormatter and GeminiMessageFormatter with matching logic. ([CYPACK-795](https://linear.app/ceedar/issue/CYPACK-795), [#846](https://github.com/ceedaragents/cyrus/pull/846))
+- Deferred TaskUpdate/TaskGet activity posting from tool_use time to tool_result time to enrich with task subject. Added `taskSubjectsByToolUseId` and `taskSubjectsById` caches to AgentSessionManager for subject resolution from TaskCreate results and TaskGet result parsing. ([CYPACK-797](https://linear.app/ceedar/issue/CYPACK-797), [#847](https://github.com/ceedaragents/cyrus/pull/847))
+
+### Added
+- Subroutine result text is now stored in procedure history when advancing between subroutines. On error results (e.g. `error_max_turns` from single-turn subroutines), `AgentSessionManager` recovers by using the last completed subroutine's result via `ProcedureAnalyzer.getLastSubroutineResult()`, allowing the procedure to continue to completion instead of failing. Added `disallowAllTools` parameter to `buildAgentRunnerConfig` and `tools` config pass-through to `ClaudeRunner` for properly disabling built-in tools. ([CYPACK-792](https://linear.app/ceedar/issue/CYPACK-792), [#843](https://github.com/ceedaragents/cyrus/pull/843))
+
+## [0.2.20] - 2026-02-05
+
+(No internal changes in this release)
+
 ## [0.2.19] - 2026-01-24
 
 ### Fixed
 
@@ -4,6 +4,86 @@ All notable changes to this project will be documented in this file.
 
 ## [Unreleased]
 
+### Added
+- GitHub PR comment support: Cyrus can now be triggered by `@cyrusagent` mentions on GitHub pull request comments, creating sessions and posting replies directly on PRs. ([CYPACK-772](https://linear.app/ceedar/issue/CYPACK-772), [#820](https://github.com/ceedaragents/cyrus/pull/820))
+
+### Fixed
+- Summary subroutines now properly disable all tools including MCP tools like Linear's create_comment ([#808](https://github.com/ceedaragents/cyrus/pull/808))
+- Procedures no longer fail when a subroutine exits with an error (e.g., hitting the max turns limit). Cyrus now recovers by using the last successful subroutine's result, allowing the workflow to continue to completion instead of stopping mid-procedure ([#818](https://github.com/ceedaragents/cyrus/pull/818))
+
+## [0.2.21] - 2026-02-09
+
+### Changed
+- **Updated Claude SDK dependencies** - Updated `@anthropic-ai/claude-agent-sdk` to v0.2.34 and `@anthropic-ai/sdk` to v0.73.0. See [claude-agent-sdk changelog](https://github.com/anthropics/claude-agent-sdk-typescript/blob/main/CHANGELOG.md#v0234) for details. ([CYPACK-788](https://linear.app/ceedar/issue/CYPACK-788), [#837](https://github.com/ceedaragents/cyrus/pull/837))
+- **Improved task and tool activity display** - Task creation now shows as concise checklist items instead of verbose multi-line entries, task status updates display the task name with status emoji, and tool search/background task output activities are now cleanly formatted. ([CYPACK-795](https://linear.app/ceedar/issue/CYPACK-795), [#846](https://github.com/ceedaragents/cyrus/pull/846))
+- **Task status updates now show task descriptions** - Task update and task detail activities now display the task subject alongside the task number (e.g., "Task #3 — Fix login bug") instead of just the number. ([CYPACK-797](https://linear.app/ceedar/issue/CYPACK-797), [#847](https://github.com/ceedaragents/cyrus/pull/847))
+
+### Fixed
+- **Procedures no longer fail when a subroutine exits with an error** - When a single-turn subroutine hits the max turns limit, Cyrus now recovers by using the last successful subroutine's result, allowing the workflow to continue to completion instead of stopping mid-procedure. ([CYPACK-792](https://linear.app/ceedar/issue/CYPACK-792), [#843](https://github.com/ceedaragents/cyrus/pull/843))
+
+### Packages
+
+#### cyrus-cloudflare-tunnel-client
+- cyrus-cloudflare-tunnel-client@0.2.21
+
+#### cyrus-config-updater
+- cyrus-config-updater@0.2.21
+
+#### cyrus-linear-event-transport
+- cyrus-linear-event-transport@0.2.21
+
+#### cyrus-claude-runner
+- cyrus-claude-runner@0.2.21
+
+#### cyrus-core
+- cyrus-core@0.2.21
+
+#### cyrus-simple-agent-runner
+- cyrus-simple-agent-runner@0.2.21
+
+#### cyrus-gemini-runner
+- cyrus-gemini-runner@0.2.21
+
+#### cyrus-edge-worker
+- cyrus-edge-worker@0.2.21
+
+#### cyrus-ai (CLI)
+- cyrus-ai@0.2.21
+
+## [0.2.20] - 2026-02-05
+
+### Fixed
+- **Agent guidance for draft PRs now respected** - When your Linear workspace guidance specifies `--draft` or requests PRs remain as drafts, Cyrus will no longer automatically convert them to ready for review. PRs also now correctly target the configured base branch instead of defaulting to main. ([CYPACK-784](https://linear.app/ceedar/issue/CYPACK-784), [#834](https://github.com/ceedaragents/cyrus/pull/834))
+
+### Packages
+
+#### cyrus-cloudflare-tunnel-client
+- cyrus-cloudflare-tunnel-client@0.2.20
+
+#### cyrus-config-updater
+- cyrus-config-updater@0.2.20
+
+#### cyrus-linear-event-transport
+- cyrus-linear-event-transport@0.2.20
+
+#### cyrus-claude-runner
+- cyrus-claude-runner@0.2.20
+
+#### cyrus-core
+- cyrus-core@0.2.20
+
+#### cyrus-simple-agent-runner
+- cyrus-simple-agent-runner@0.2.20
+
+#### cyrus-gemini-runner
+- cyrus-gemini-runner@0.2.20
+
+#### cyrus-edge-worker
+- cyrus-edge-worker@0.2.20
+
+#### cyrus-ai (CLI)
+- cyrus-ai@0.2.20
+
 ## [0.2.19] - 2026-01-24
 
 ### Fixed
 
@@ -1,6 +1,6 @@
 {
 	"name": "cyrus-ai",
-	"version": "0.2.19",
+	"version": "0.2.21",
 	"description": "AI-powered Linear issue automation using Claude",
 	"main": "dist/src/app.js",
 	"types": "dist/src/app.d.ts",
 
@@ -1,132 +1,80 @@
-/**
- * Log levels in order of severity
- */
-export enum LogLevel {
-	DEBUG = 0,
-	INFO = 1,
-	SUCCESS = 2,
-	WARN = 3,
-	ERROR = 4,
-	SILENT = 5,
-}
+import {
+	createLogger,
+	type ILogger,
+	type LogContext,
+	type LogLevel,
+} from "cyrus-core";
+
+// Re-export LogLevel from cyrus-core so existing consumers don't break
+export { LogLevel } from "cyrus-core";
 
 /**
  * Logger configuration options
  */
 export interface LoggerOptions {
 	/** Minimum log level to output */
 	level?: LogLevel;
-	/** Prefix to add to all log messages */
+	/** Prefix to add to all log messages (used as component name) */
 	prefix?: string;
 	/** Whether to include timestamps */
 	timestamps?: boolean;
 }
 
 /**
- * Simple, zero-dependency logger service with structured logging
+ * CLI-specific logger that wraps the core ILogger.
+ *
+ * Provides CLI-presentation features (emoji formatting, raw output,
+ * dividers, child loggers) on top of the standard core logging interface.
+ *
+ * Implements ILogger so it can be passed to packages that expect the core interface.
  */
-export class Logger {
-	private level: LogLevel;
+export class Logger implements ILogger {
+	private coreLogger: ILogger;
 	private prefix: string;
 	private timestamps: boolean;
 
 	constructor(options: LoggerOptions = {}) {
-		this.level = options.level ?? this.getLogLevelFromEnv();
 		this.prefix = options.prefix ?? "";
 		this.timestamps = options.timestamps ?? false;
-	}
-
-	/**
-	 * Get log level from environment variable
-	 */
-	private getLogLevelFromEnv(): LogLevel {
-		const envLevel = process.env.CYRUS_LOG_LEVEL?.toUpperCase();
-		switch (envLevel) {
-			case "DEBUG":
-				return LogLevel.DEBUG;
-			case "INFO":
-				return LogLevel.INFO;
-			case "SUCCESS":
-				return LogLevel.SUCCESS;
-			case "WARN":
-				return LogLevel.WARN;
-			case "ERROR":
-				return LogLevel.ERROR;
-			case "SILENT":
-				return LogLevel.SILENT;
-			default:
-				return LogLevel.INFO;
-		}
-	}
-
-	/**
-	 * Format a log message with optional prefix and timestamp
-	 */
-	private format(message: string): string {
-		let formatted = message;
-
-		if (this.prefix) {
-			formatted = `[${this.prefix}] ${formatted}`;
-		}
-
-		if (this.timestamps) {
-			const timestamp = new Date().toISOString();
-			formatted = `${timestamp} ${formatted}`;
-		}
-
-		return formatted;
-	}
-
-	/**
-	 * Check if a log level should be output
-	 */
-	private shouldLog(level: LogLevel): boolean {
-		return level >= this.level;
+		this.coreLogger = createLogger({
+			component: this.prefix || "CLI",
+			level: options.level,
+		});
 	}
 
 	/**
 	 * Debug log (lowest priority)
 	 */
 	debug(message: string, ...args: any[]): void {
-		if (this.shouldLog(LogLevel.DEBUG)) {
-			console.log(this.format(`🔍 ${message}`), ...args);
-		}
+		this.coreLogger.debug(message, ...args);
 	}
 
 	/**
 	 * Info log (normal priority)
 	 */
 	info(message: string, ...args: any[]): void {
-		if (this.shouldLog(LogLevel.INFO)) {
-			console.log(this.format(message), ...args);
-		}
+		this.coreLogger.info(message, ...args);
 	}
 
 	/**
-	 * Success log (positive outcome)
+	 * Success log - maps to info level with check mark prefix
 	 */
 	success(message: string, ...args: any[]): void {
-		if (this.shouldLog(LogLevel.SUCCESS)) {
-			console.log(this.format(`✅ ${message}`), ...args);
-		}
+		this.coreLogger.info(message, ...args);
 	}
 
 	/**
 	 * Warning log
 	 */
 	warn(message: string, ...args: any[]): void {
-		if (this.shouldLog(LogLevel.WARN)) {
-			console.warn(this.format(`⚠️  ${message}`), ...args);
-		}
+		this.coreLogger.warn(message, ...args);
 	}
 
 	/**
 	 * Error log (highest priority)
 	 */
 	error(message: string, ...args: any[]): void {
-		if (this.shouldLog(LogLevel.ERROR)) {
-			console.error(this.format(`❌ ${message}`), ...args);
-		}
+		this.coreLogger.error(message, ...args);
 	}
 
 	/**
@@ -141,7 +89,7 @@ export class Logger {
 	 */
 	child(prefix: string): Logger {
 		return new Logger({
-			level: this.level,
+			level: this.coreLogger.getLevel(),
 			prefix: this.prefix ? `${this.prefix}:${prefix}` : prefix,
 			timestamps: this.timestamps,
 		});
@@ -151,21 +99,29 @@ export class Logger {
 	 * Print a divider line
 	 */
 	divider(length = 70): void {
-		this.raw("─".repeat(length));
+		this.raw("\u2500".repeat(length));
+	}
+
+	/**
+	 * Create a new logger with additional context.
+	 * Delegates to the core logger's withContext.
+	 */
+	withContext(context: LogContext): ILogger {
+		return this.coreLogger.withContext(context);
 	}
 
 	/**
 	 * Set log level dynamically
 	 */
 	setLevel(level: LogLevel): void {
-		this.level = level;
+		this.coreLogger.setLevel(level);
 	}
 
 	/**
 	 * Get current log level
 	 */
 	getLevel(): LogLevel {
-		return this.level;
+		return this.coreLogger.getLevel();
 	}
 }
 
 
@@ -1,6 +1,6 @@
 {
 	"name": "cyrus-claude-runner",
-	"version": "0.2.19",
+	"version": "0.2.21",
 	"description": "Claude CLI execution wrapper for Cyrus",
 	"type": "module",
 	"main": "dist/index.js",
@@ -16,8 +16,8 @@
 		"typecheck": "tsc --noEmit"
 	},
 	"dependencies": {
-		"@anthropic-ai/claude-agent-sdk": "^0.2.7",
-		"@anthropic-ai/sdk": "^0.71.2",
+		"@anthropic-ai/claude-agent-sdk": "^0.2.34",
+		"@anthropic-ai/sdk": "^0.73.0",
 		"@linear/sdk": "^64.0.0",
 		"cyrus-core": "workspace:*",
 		"dotenv": "^16.4.5",
Original file line number	Diff line number	Diff line change
`@@ -1,6 +1,6 @@`
`1`	`1`	`{`
`2`	`2`	`"name": "cyrus-ai",`
`3`		`- "version": "0.2.19",`
	`3`	`+ "version": "0.2.21",`
`4`	`4`	`"description": "AI-powered Linear issue automation using Claude",`
`5`	`5`	`"main": "dist/src/app.js",`
`6`	`6`	`"types": "dist/src/app.d.ts",`