Cache Copilot token

[notlikejuice]

Summary

• cache GitHub Copilot auth token to disk

Testing

• npm run lint
• npm test

---

https://chatgpt.com/codex/tasks/task_e_684f6dfde9fc8326a69a2fe6d88940e0

[Copilot]

Pull Request Overview

This PR adds on-disk caching for the GitHub Copilot authentication token to avoid unnecessary network requests.

• Reads and reuses a valid token from ~/.codex/copilot-token.json if not expired
• Persists newly fetched tokens with secure file permissions
• Imports fs, os, and path and defines a constant cache file path

Comments suppressed due to low confidence (2)

codex-cli/src/utils/openai-client.ts:91

• [nitpick] The variable name disk is ambiguous. Consider renaming it to something like cachedTokenData or diskCache to clarify its purpose.

const disk = JSON.parse(fs.readFileSync(COPILOT_TOKEN_FILE, "utf-8"));

codex-cli/src/utils/openai-client.ts:90

• The new disk caching logic isn’t covered by existing tests. Please add unit tests that simulate fresh fetch, valid cache reuse, and expired cache scenarios.

try {

[Copilot]

This uses synchronous file I/O within an async function, which can block the event loop. Consider using fs.promises.readFile or other async APIs to avoid blocking.

Suggested change

	const disk = JSON.parse(fs.readFileSync(COPILOT_TOKEN_FILE, "utf-8"));
	const disk = JSON.parse(await fs.promises.readFile(COPILOT_TOKEN_FILE, "utf-8"));

[Copilot]

Silently swallowing all errors may make debugging difficult. Consider logging or handling specific error cases to aid in diagnosing cache read failures.

Suggested change

	} catch {
	/* ignore */
	} catch (error) {
	console.error("Failed to read Copilot token from disk:", error);