Skip to content

Comments

Update log file path to account for oz subdirectory for CLI logs.#8

Open
abhishekp106 wants to merge 1 commit intomainfrom
abhishek/update-log-path-for-oz-subdirectory
Open

Update log file path to account for oz subdirectory for CLI logs.#8
abhishekp106 wants to merge 1 commit intomainfrom
abhishek/update-log-path-for-oz-subdirectory

Conversation

@abhishekp106
Copy link

WISOTT! In conjunction with https://github.com/warpdotdev/warp-internal/pull/22626.

Co-Authored-By: Oz oz-agent@warp.dev

CLI logs are now written to an 'oz' subdirectory within the Warp
state directory to avoid evicting GUI logs during rotation.

Co-Authored-By: Oz <oz-agent@warp.dev>
Copy link
Author

This stack of pull requests is managed by Graphite. Learn more about stacking.

@abhishekp106 abhishekp106 changed the title Update log file path to include 'oz' subdirectory Update log file path to account for 'oz' subdirectory for CLI logs. Feb 24, 2026
@abhishekp106 abhishekp106 changed the title Update log file path to account for 'oz' subdirectory for CLI logs. Update log file path to account for oz subdirectory for CLI logs. Feb 24, 2026
const channelSuffix = channel === 'stable' ? '' : `-${channel}`;
const logFileName = channel === 'stable' ? 'warp.log' : `warp_${channel}.log`;
const warpLogPath = path.join(stateDir, `warp-terminal${channelSuffix}`, logFileName);
const warpLogPath = path.join(stateDir, `warp-terminal${channelSuffix}`, 'oz', logFileName);
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The action allows you to pick the version of Oz that you want to install so there is a chance that someone runs the newest version of the action with an older version of Oz that doesn't yet write to this directory.

I don't think it's likely to happen and probably not worth writing code around but perhaps worth a comment in the code for posterity?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants