jedarden/FABRIC
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

fix(ci): handle missing homedir in ThemeManager for CI environments

Browse source 
Wrap os.homedir() in try/catch and guard against undefined return value
in ThemeManager.getConfigPath(). In CI environments, os.homedir() may be
undefined, causing path.join() to throw and preventing SessionReplay
from being constructed. When homedir is unavailable, theme persistence
is gracefully skipped and the default dark theme is used.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
This commit is contained in:
jedarden 2026-03-19 07:38:40 -04:00
parent d35c9adcdc
commit 8434701bd5
1 changed files with 15 additions and 5 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

20
src/tui/utils/theme.ts
View file

@ -189,11 +189,20 @@ export class ThemeManager {
* Get the config file path for theme persistence
*/
private getConfigPath(): string {
const configDir = path.join(os.homedir(), '.fabric');
if (!fs.existsSync(configDir)) {
fs.mkdirSync(configDir, { recursive: true });
try {
const home = os.homedir();
if (!home) {
return '';
}
const configDir = path.join(home, '.fabric');
if (!fs.existsSync(configDir)) {
fs.mkdirSync(configDir, { recursive: true });
}
return path.join(configDir, 'theme.json');
} catch {
// In CI or environments without a home directory, skip config persistence
return '';
}
return path.join(configDir, 'theme.json');
}
/**
@ -201,7 +210,7 @@ export class ThemeManager {
*/
private loadTheme(): ThemeName {
try {
if (fs.existsSync(this.configPath)) {
if (this.configPath && fs.existsSync(this.configPath)) {
const content = fs.readFileSync(this.configPath, 'utf-8');
const config = JSON.parse(content);
if (config.theme === 'dark' || config.theme === 'light') {
@ -218,6 +227,7 @@ export class ThemeManager {
* Save theme to config file
*/
private saveTheme(): void {
if (!this.configPath) return;
try {
const config = { theme: this.currentTheme };
fs.writeFileSync(this.configPath, JSON.stringify(config, null, 2), 'utf-8');