copilot-relay 0.2.2 → 0.2.4

package/README.md

@@ -36,7 +36,7 @@ Routing:
 
 | Requested model | Upstream model |
 | --- | --- |
-| contains `opus` | `claude-opus-4.8` |
+| contains `opus` | `claude-opus-5` |
 | `gpt-5.6-sol[1m]`, plain `gpt-5.6-sol`, or another non-Opus alias | `gpt-5.6-sol` |
 
 The relay advertises `gpt-5.6-sol[1m]` to Claude Code. The `[1m]` selector
@@ -80,6 +80,11 @@ upstreamTimeoutSeconds: 180
 webSearchBackend:
 ```
 
+copilot-relay writes the resolved config back to this file, so every key is
+present after the first start. Shipped defaults therefore apply to fresh
+installs only — once a value is in your `config.yaml` it is never rewritten by
+an upgrade. To pick up a changed default, edit the key yourself.
+
 `logLevel` controls verbosity:
 
 | Level | Logs |
@@ -119,13 +124,17 @@ Upstream failures are logged at `error` with full request and response context i
 Unsupported Claude API requests are logged at `error` with the local method/path
 and detailed request payload.
 
-Logs are written to `~/.copilot-relay/logs/copilot-relay.log`; old `.log` files are cleaned according to `logRetentionDays`.
+Logs are written to `~/.copilot-relay/logs/copilot-relay.<local-date>.log`. The
+active file rotates at local midnight, and files older than `logRetentionDays`
+local calendar days are deleted. Rotation is what makes retention take effect:
+without it the single log file's mtime was refreshed by every append, so nothing
+ever aged out.
 
 Quick inspection:
 
 ```sh
-tail -f ~/.copilot-relay/logs/copilot-relay.log
-grep -n "Failed to create\\|Startup preflight failed" ~/.copilot-relay/logs/copilot-relay.log
+tail -f ~/.copilot-relay/logs/copilot-relay.$(date +%F).log
+grep -n "Failed to create\\|Startup preflight failed" ~/.copilot-relay/logs/copilot-relay.*.log
 ```
 
 See [`docs/troubleshooting.md`](docs/troubleshooting.md) for common debugging workflows.

package/config.default.yaml

@@ -4,6 +4,10 @@
 # Edit the user config file, not this template, for local changes.
 # Most values hot-reload while the service is running. host and port still
 # require restart because the listening socket is already bound.
+#
+# Values here apply to a fresh install only. copilot-relay writes the resolved
+# config back to disk, so once a key exists in your config.yaml it is yours:
+# changing a shipped default never rewrites it.
 
 # Local host for the Claude Code-compatible HTTP server.
 host: 127.0.0.1
@@ -29,6 +33,9 @@ claudeSetup: true
 logLevel: info
 
 # Number of days to keep files in ~/.copilot-relay/logs.
+# Counted in local calendar days: 3 keeps today plus the previous two days.
+# The active log file is copilot-relay.<local-date>.log and rotates at local
+# midnight, which is what makes this setting take effect at all.
 logRetentionDays: 3
 
 # Default upstream thinking/reasoning effort sent to Copilot.
@@ -48,4 +55,4 @@ webSearchBackend:
 gptModel: gpt-5.6-sol
 
 # Upstream Copilot model used for requests containing "opus".
-opusModel: claude-opus-4.8
+opusModel: claude-opus-5

package/dist/main.js

@@ -57,8 +57,26 @@ const configPath = path.join(appDir, "config.yaml");
 const copilotTokenPath = path.join(appDir, "copilot_token.json");
 const githubTokenPath = path.join(appDir, "github_token");
 const logsDir = path.join(appDir, "logs");
-const logPath = path.join(logsDir, "copilot-relay.log");
+const logFileBaseName = "copilot-relay";
+const legacyLogPath = path.join(logsDir, `${logFileBaseName}.log`);
 const pidPath = path.join(appDir, "copilot-relay.pid");
+/**
+* Local-date stamp used in log file names.
+*
+* Local rather than UTC on purpose: `logRetentionDays` is a human-facing "how
+* many days do I keep" setting, and a UTC stamp would roll the active file over
+* in the middle of the local afternoon for anyone west of Greenwich.
+*/
+const formatLogDate = (date) => [
+	date.getFullYear(),
+	String(date.getMonth() + 1).padStart(2, "0"),
+	String(date.getDate()).padStart(2, "0")
+].join("-");
+/**
+* Path of the log file that owns `date`. Resolved per write rather than cached,
+* so a long-running relay rotates at local midnight with no timer to drift.
+*/
+const getLogPath = (date = /* @__PURE__ */ new Date()) => path.join(logsDir, `${logFileBaseName}.${formatLogDate(date)}.log`);
 const paths = {
 	appDir,
 	configPath,
@@ -70,7 +88,8 @@ const paths = {
 		path.join(homeDir, ".copilot-tennel.yaml")
 	],
 	legacyCopilotTokenPaths,
-	logPath,
+	legacyLogPath,
+	logFileBaseName,
 	logsDir,
 	pidPath
 };
@@ -103,8 +122,10 @@ async function ensureFile(filePath, options = {}) {
 //#endregion
 //#region src/lib/log.ts
 const logCleanupCheckIntervalMs = 3600 * 1e3;
+const millisecondsPerDay = 1440 * 60 * 1e3;
 let logRetentionDays = 3;
 let nextLogCleanupCheckAt = 0;
+const datedLogFilePattern = /* @__PURE__ */ new RegExp(`^${paths.logFileBaseName}\\.(\\d{4})-(\\d{2})-(\\d{2})\\.log$`);
 const consolaLevelByName = {
 	error: 0,
 	info: 3,
@@ -120,7 +141,22 @@ const setLogLevel = (level) => {
 	currentLogLevel = consolaLevelByName[level];
 	consola.level = consolaLevelByName[level];
 };
-const formatLogValue = (value) => typeof value === "string" ? value : inspect(value, { depth: null });
+/**
+* Render one logged value as a single physical line.
+*
+* `breakLength: Infinity` matters as much as the depth cap. The previous
+* `inspect(value, { depth: null })` pretty-printed each payload across
+* thousands of physical lines, which was both the dominant source of log volume
+* and the reason the `grep` recipes in docs/logging.md returned a fragment of an
+* object instead of the matching entry. One entry is now one line.
+*/
+const formatLogValue = (value) => typeof value === "string" ? value : inspect(value, {
+	breakLength: Infinity,
+	compact: true,
+	depth: 6,
+	maxArrayLength: 100,
+	maxStringLength: 4e3
+});
 const writeLogFile = async (level, values) => {
 	await fs.mkdir(paths.logsDir, { recursive: true });
 	await cleanupLogsIfDue();
@@ -129,7 +165,7 @@ const writeLogFile = async (level, values) => {
 		level,
 		values.map(formatLogValue).join(" ")
 	].join(" ");
-	await fs.appendFile(paths.logPath, `${line}\n`);
+	await fs.appendFile(getLogPath(), `${line}\n`);
 };
 const cleanupLogsIfDue = async () => {
 	const now = Date.now();
@@ -144,14 +180,31 @@ const wrapFileLog = (level, fn) => ((...args) => {
 consola.error = wrapFileLog("error", consola.error.bind(consola));
 consola.info = wrapFileLog("info", consola.info.bind(consola));
 consola.debug = wrapFileLog("debug", consola.debug.bind(consola));
+/**
+* Calendar day a log file belongs to, or undefined when its name carries no
+* date stamp.
+*
+* The filename stamp is preferred over mtime because mtime is rewritten by
+* backups, `cp`, and editors touching the file, any of which would silently
+* extend or shorten the retention window.
+*/
+const parseLogFileDate = (fileName) => {
+	const match = datedLogFilePattern.exec(fileName);
+	if (!match) return;
+	const [, year, month, day] = match;
+	const date = new Date(Number(year), Number(month) - 1, Number(day));
+	return date.getFullYear() === Number(year) && date.getMonth() === Number(month) - 1 && date.getDate() === Number(day) ? date : void 0;
+};
 const cleanupLogs = async (retentionDays) => {
 	logRetentionDays = retentionDays;
 	await fs.mkdir(paths.logsDir, { recursive: true });
-	const cutoff = Date.now() - retentionDays * 24 * 60 * 60 * 1e3;
+	const startOfToday = /* @__PURE__ */ new Date();
+	startOfToday.setHours(0, 0, 0, 0);
+	const cutoff = startOfToday.getTime() - (retentionDays - 1) * millisecondsPerDay;
 	const entries = await fs.readdir(paths.logsDir, { withFileTypes: true });
 	await Promise.all(entries.filter((entry) => entry.isFile() && entry.name.endsWith(".log")).map(async (entry) => {
 		const filePath = `${paths.logsDir}/${entry.name}`;
-		if ((await fs.stat(filePath)).mtimeMs < cutoff) await fs.unlink(filePath);
+		if ((parseLogFileDate(entry.name)?.getTime() ?? (await fs.stat(filePath)).mtimeMs) < cutoff) await fs.rm(filePath, { force: true });
 	}));
 };
 const log = consola;
@@ -346,7 +399,7 @@ const runtimeState = {};
 const defaultReasoningEffort = "max";
 const defaultModelRouting = {
 	gptModel: "gpt-5.6-sol",
-	opusModel: "claude-opus-4.8"
+	opusModel: "claude-opus-5"
 };
 const isReasoningEffort = (value) => value === "none" || value === "low" || value === "medium" || value === "high" || value === "xhigh" || value === "max";
 const exactGpt56SolPattern = /^gpt-5\.6-sol(?:\[1m\])*$/i;
@@ -382,6 +435,14 @@ const logLevels = [
 	"info",
 	"debug"
 ];
+/**
+* Values used when a key is absent from the user's config.
+*
+* readAppConfig() writes the resolved config back to disk, so an existing
+* install has every key materialized and never consults these again. Changing
+* one therefore affects fresh installs only, by design: copilot-relay does not
+* rewrite a value the user's config already holds.
+*/
 const defaultConfig = {
 	claudeSetup: true,
 	copilotBaseUrl: "https://api.githubcopilot.com",
@@ -389,7 +450,7 @@ const defaultConfig = {
 	host: "127.0.0.1",
 	logLevel: "info",
 	logRetentionDays: 3,
-	opusModel: "claude-opus-4.8",
+	opusModel: "claude-opus-5",
 	port: 4142,
 	thinkEffort: defaultReasoningEffort,
 	upstreamTimeoutSeconds: 180,