geekbot-cli 0.2.2 → 0.2.3

package/package.json

@@ -1,6 +1,6 @@
 {
 	"name": "geekbot-cli",
-	"version": "0.2.2",
+	"version": "0.2.3",
 	"description": "CLI tool for managing Geekbot standups, reports, and polls — designed for AI agents and humans",
 	"type": "module",
 	"license": "MIT",

package/src/auth/resolver.ts

@@ -13,13 +13,19 @@ export async function resolveCredential(
 ): Promise<CredentialResult> {
 	// Priority 1: --api-key flag
 	if (options.apiKeyFlag) {
-		return { apiKey: options.apiKeyFlag.trim(), source: "flag" };
+		const trimmed = options.apiKeyFlag.trim();
+		if (trimmed) {
+			return { apiKey: trimmed, source: "flag" };
+		}
 	}
 
 	// Priority 2: GEEKBOT_API_KEY env var
 	const envKey = process.env.GEEKBOT_API_KEY;
 	if (envKey) {
-		return { apiKey: envKey.trim(), source: "env" };
+		const trimmed = envKey.trim();
+		if (trimmed) {
+			return { apiKey: trimmed, source: "env" };
+		}
 	}
 
 	// Priority 3: OS keychain

package/src/cli/commands/report.ts

@@ -11,8 +11,8 @@ export function createReportCommand(): Command {
 		.description("List reports with optional filters")
 		.option("--standup-id <id>", "Filter by standup ID")
 		.option("--user-id <id>", "Filter by user ID")
-		.option("--before <date>", "Reports before date (ISO 8601 or unix timestamp)")
-		.option("--after <date>", "Reports after date (ISO 8601 or unix timestamp)")
+		.option("--before <date>", "Reports before date (YYYY-MM-DD or unix timestamp)")
+		.option("--after <date>", "Reports after date (YYYY-MM-DD or unix timestamp)")
 		.option("--limit <n>", "Max number of reports to return")
 		.addHelpText(
 			"after",

package/src/handlers/standup-handlers.ts

@@ -99,6 +99,18 @@ async function enrichNotFound(
 		await fn(client);
 	} catch (error) {
 		if (error instanceof CliError && error.code === "not_found") {
+			// Don't enrich user-not-member errors with standup-ID suggestions
+			const isUserNotMember = /user is not member/i.test(error.message);
+			if (isUserNotMember) {
+				throw new CliError(
+					error.message,
+					error.code,
+					error.exitCode,
+					error.retryable,
+					"The specified user is not a member of this standup. Check members with: geekbot standup get <id>",
+					error.context,
+				);
+			}
 			const suggestion = await buildNotFoundSuggestion(client, resourceType);
 			if (suggestion) {
 				throw new CliError(
@@ -117,11 +129,14 @@ async function enrichNotFound(
 
 // ── Handlers ──────────────────────────────────────────────────────────
 
-/** Brief standup projection — only essential fields for discovery */
+/** Brief standup projection — essential fields for discovery */
 export interface StandupBrief {
 	id: number;
 	name: string;
 	channel: string;
+	time: string;
+	timezone: string;
+	days: string[];
 }
 
 /**
@@ -173,6 +188,9 @@ export async function handleStandupList(
 			id: s.id,
 			name: s.name,
 			channel: s.channel,
+			time: s.time,
+			timezone: s.timezone,
+			days: s.days,
 		}));
 		writeOutput(successList(brief));
 		return;
@@ -352,9 +370,12 @@ export async function handleStandupReplace(
 			const previousStandup = StandupSchema.parse(prevRaw);
 
 			// Build full body — PUT requires complete representation
-			const time = options.time ?? "10:00";
+			// Carry forward from previous standup when flags are omitted
+			const time = options.time ?? previousStandup.time.slice(0, 5);
 			validateTimeFormat(time);
-			const days = validateDayAbbreviations((options.days ?? "Mon,Tue,Wed,Thu,Fri").split(","));
+			const days = options.days
+				? validateDayAbbreviations(options.days.split(","))
+				: previousStandup.days;
 
 			const body: Record<string, unknown> = {
 				name: options.name,
@@ -363,9 +384,7 @@ export async function handleStandupReplace(
 				days,
 			};
 
-			if (options.timezone !== undefined) {
-				body.timezone = options.timezone;
-			}
+			body.timezone = options.timezone ?? previousStandup.timezone;
 
 			// questions: use provided or carry forward from existing standup
 			if (options.questions !== undefined) {
@@ -378,11 +397,14 @@ export async function handleStandupReplace(
 				body.users = validateSlackIdList(options.users, "user ID");
 				body.sync_channel_members = false;
 			} else {
-				body.sync_channel_members = true;
+				body.users = previousStandup.users.map((u) => u.id);
+				body.sync_channel_members = previousStandup.sync_channel_members ?? false;
 			}
 
 			if (options.waitTime !== undefined) {
 				body.wait_time = validateWaitTime(options.waitTime);
+			} else {
+				body.wait_time = previousStandup.wait_time;
 			}
 
 			const raw = await client.put<unknown>(`/v1/standups/${numericId}`, body);
@@ -522,10 +544,14 @@ function buildReplaceUndoCommand(id: number, prev: Standup): string {
 		parts.push(`--days ${shellEscape(prev.days.join(","))}`);
 	}
 
-	if (prev.wait_time > 0) {
+	if (prev.wait_time !== 0) {
 		parts.push(`--wait-time ${prev.wait_time}`);
 	}
 
+	if (prev.users.length > 0) {
+		parts.push(`--users ${prev.users.map((u) => u.id).join(",")}`);
+	}
+
 	if (prev.questions.length > 0) {
 		parts.push(`--questions ${shellEscape(JSON.stringify(prev.questions.map((q) => q.text)))}`);
 	}

package/src/schemas/report.ts

@@ -6,7 +6,7 @@ const ReportAnswerSchema = z.object({
 	id: z.number(),
 	question: z.string().optional().default(""),
 	question_id: z.number().optional(),
-	answer: z.string(),
+	answer: z.string().nullable(),
 	answer_type: z.string().optional().default("text"),
 	images: z
 		.array(
@@ -34,7 +34,7 @@ const TimelineReportRawSchema = z.object({
 	slack_ts: z.string().nullable().optional(),
 	channel: z.string().optional().default(""),
 	questions: z.array(ReportAnswerSchema),
-	member: CompactUserSchema,
+	member: CompactUserSchema.optional(),
 	is_anonymous: z.boolean().optional().default(false),
 	broadcast_thread: z.boolean().optional().default(false),
 	is_confidential: z.boolean().optional().default(false),
@@ -49,11 +49,11 @@ const TimelineReportRawSchema = z.object({
 const SubmittedReportRawSchema = z.object({
 	id: z.number(),
 	standup_id: z.number(),
-	timestamp: z.number(),
+	timestamp: z.number().nullable(),
 	slack_ts: z.string().nullable().optional(),
 	started_at: z.number().optional(),
 	done_at: z.number().optional(),
-	broadcasted_at: z.number().nullable().optional(),
+	broadcasted_at: z.string().nullable().optional(),
 	channel: z.string().optional().default(""),
 	member: z
 		.object({
@@ -89,7 +89,7 @@ function normalizeTimelineReport(raw: z.output<typeof TimelineReportRawSchema>):
 		standup_id: raw.standup_id,
 		created_at: new Date(raw.timestamp * 1000).toISOString(),
 		questions: raw.questions,
-		member: raw.member, // Already normalized by CompactUserSchema (profileImg -> profile_img)
+		member: raw.member ?? null, // Already normalized by CompactUserSchema (profileImg -> profile_img)
 		is_anonymous: raw.is_anonymous,
 		standup_name: raw.standup_name,
 	};
@@ -103,7 +103,7 @@ export const SubmittedReportSchema = SubmittedReportRawSchema.transform(
 	(raw): Report => ({
 		id: raw.id,
 		standup_id: raw.standup_id,
-		created_at: new Date(raw.timestamp * 1000).toISOString(),
+		created_at: new Date((raw.timestamp ?? raw.done_at ?? Date.now() / 1000) * 1000).toISOString(),
 		questions: raw.answers, // POST uses "answers" key, normalize to "questions"
 		member: raw.member
 			? {

package/src/schemas/standup.ts

@@ -50,7 +50,7 @@ const StandupRawSchema = z.object({
 
 /** Normalize a raw standup: convert wait_time from seconds to minutes */
 function normalizeStandup(raw: z.output<typeof StandupRawSchema>) {
-	return { ...raw, wait_time: raw.wait_time / 60 }; // NORM: seconds -> minutes (Pitfall 1)
+	return { ...raw, wait_time: raw.wait_time === -1 ? -1 : raw.wait_time / 60 }; // NORM: seconds -> minutes; -1 is "exact time" sentinel
 }
 
 /** Normalized standup -- wait_time converted from seconds to minutes */

package/src/utils/input-parsers.ts

@@ -197,7 +197,7 @@ export function parseDateFilter(raw: string, label: string): string {
 			"validation_error",
 			ExitCode.VALIDATION,
 			false,
-			"Accepted formats: ISO 8601 (2024-01-15) or unix timestamp (1705276800)",
+			"Accepted formats: YYYY-MM-DD (2024-01-15) or unix timestamp (1705276800)",
 		);
 	}
 
@@ -208,7 +208,7 @@ export function parseDateFilter(raw: string, label: string): string {
 			"validation_error",
 			ExitCode.VALIDATION,
 			false,
-			"Accepted formats: ISO 8601 (2024-01-15) or unix timestamp (1705276800)",
+			"Accepted formats: YYYY-MM-DD (2024-01-15) or unix timestamp (1705276800)",
 		);
 	}
 
@@ -226,7 +226,7 @@ export function parseDateFilter(raw: string, label: string): string {
 			"validation_error",
 			ExitCode.VALIDATION,
 			false,
-			"Accepted formats: ISO 8601 (2024-01-15) or unix timestamp (1705276800)",
+			"Accepted formats: YYYY-MM-DD (2024-01-15) or unix timestamp (1705276800)",
 		);
 	}
 

package/src/utils/receipt.ts

@@ -41,10 +41,14 @@ export function buildDeleteUndoCommand(standup: Standup): string {
 		parts.push(`--days ${shellEscape(standup.days.join(","))}`);
 	}
 
-	if (standup.wait_time > 0) {
+	if (standup.wait_time !== 0) {
 		parts.push(`--wait-time ${standup.wait_time}`);
 	}
 
+	if (standup.users.length > 0) {
+		parts.push(`--users ${standup.users.map((u) => u.id).join(",")}`);
+	}
+
 	if (standup.questions.length > 0) {
 		parts.push(`--questions ${shellEscape(JSON.stringify(standup.questions.map((q) => q.text)))}`);
 	}
@@ -60,6 +64,8 @@ const FIELD_TO_FLAG: Record<string, string> = {
 	timezone: "--timezone",
 	days: "--days",
 	wait_time: "--wait-time",
+	users: "--users",
+	questions: "--questions",
 };
 
 /**
@@ -83,6 +89,12 @@ export function buildUpdateUndoCommand(
 			parts.push(`${flag} ${shellEscape(prevValue.slice(0, 5))}`);
 		} else if (key === "days" && Array.isArray(prevValue)) {
 			parts.push(`${flag} ${shellEscape(prevValue.join(","))}`);
+		} else if (key === "users" && Array.isArray(prevValue)) {
+			parts.push(`${flag} ${prevValue.map((u: { id: string }) => u.id).join(",")}`);
+		} else if (key === "questions" && Array.isArray(prevValue)) {
+			parts.push(
+				`${flag} ${shellEscape(JSON.stringify(prevValue.map((q: { text: string }) => q.text)))}`,
+			);
 		} else if (typeof prevValue === "number") {
 			parts.push(`${flag} ${prevValue}`);
 		} else if (typeof prevValue === "string") {

package/src/utils/validation.ts

@@ -70,13 +70,14 @@ export function validateSlackIdList(value: string, label: string): string[] {
  */
 export function validateWaitTime(value: string): number {
 	const num = Number(value);
+	if (num === -1) return num; // -1 is "exact time" sentinel
 	if (!Number.isSafeInteger(num) || num < 0) {
 		throw new CliError(
-			`Invalid wait time: "${value}". Must be a non-negative integer.`,
+			`Invalid wait time: "${value}". Must be a non-negative integer or -1 for exact time.`,
 			"validation_error",
 			ExitCode.VALIDATION,
 			false,
-			`Provide a numeric value in minutes, e.g.: --wait-time 15`,
+			`Provide a numeric value in minutes, e.g.: --wait-time 15 (or -1 for exact time)`,
 		);
 	}
 	return num;