fix(webkit): add directory verification logging for gc-safe path handling
- commit
- bba942658de7b9a2e423033104f55d6ccf8f94c4
- author
- bnema · · 2025-12-18 04:47 UTC
- parent
- a59671e
- signature
- unsigned
2 files · +11 -1
M go.mod +1 -1
| @@ -13,6 +13,7 @@ require ( | |||
| 13 | 13 | github.com/pressly/goose/v3 v3.26.0 | |
| 14 | 14 | github.com/rs/zerolog v1.33.0 | |
| 15 | 15 | github.com/sahilm/fuzzy v0.1.1 | |
| 16 | + | github.com/spf13/cobra v1.10.2 | |
| 16 | 17 | github.com/spf13/viper v1.21.0 | |
| 17 | 18 | github.com/stretchr/testify v1.11.1 | |
| 18 | 19 | ) | |
| @@ -47,7 +48,6 @@ require ( | |||
| 47 | 48 | github.com/sourcegraph/conc v0.3.1-0.20240121214520-5f936abd7ae8 // indirect | |
| 48 | 49 | github.com/spf13/afero v1.15.0 // indirect | |
| 49 | 50 | github.com/spf13/cast v1.10.0 // indirect | |
| 50 | - | github.com/spf13/cobra v1.10.2 // indirect | |
| 51 | 51 | github.com/spf13/pflag v1.0.10 // indirect | |
| 52 | 52 | github.com/stretchr/objx v0.5.2 // indirect | |
| 53 | 53 | github.com/subosito/gotenv v1.6.0 // indirect | |
| @@ -13,6 +13,7 @@ require ( | |||||
| 13 | github.com/pressly/goose/v3 v3.26.0 | 13 | github.com/pressly/goose/v3 v3.26.0 | ||
| 14 | github.com/rs/zerolog v1.33.0 | 14 | github.com/rs/zerolog v1.33.0 | ||
| 15 | github.com/sahilm/fuzzy v0.1.1 | 15 | github.com/sahilm/fuzzy v0.1.1 | ||
| 16 | + | github.com/spf13/cobra v1.10.2 | |||
| 16 | github.com/spf13/viper v1.21.0 | 17 | github.com/spf13/viper v1.21.0 | ||
| 17 | github.com/stretchr/testify v1.11.1 | 18 | github.com/stretchr/testify v1.11.1 | ||
| 18 | ) | 19 | ) | ||
| @@ -47,7 +48,6 @@ require ( | |||||
| 47 | github.com/sourcegraph/conc v0.3.1-0.20240121214520-5f936abd7ae8 // indirect | 48 | github.com/sourcegraph/conc v0.3.1-0.20240121214520-5f936abd7ae8 // indirect | ||
| 48 | github.com/spf13/afero v1.15.0 // indirect | 49 | github.com/spf13/afero v1.15.0 // indirect | ||
| 49 | github.com/spf13/cast v1.10.0 // indirect | 50 | github.com/spf13/cast v1.10.0 // indirect | ||
| 50 | - | github.com/spf13/cobra v1.10.2 // indirect | |||
| 51 | github.com/spf13/pflag v1.0.10 // indirect | 51 | github.com/spf13/pflag v1.0.10 // indirect | ||
| 52 | github.com/stretchr/objx v0.5.2 // indirect | 52 | github.com/stretchr/objx v0.5.2 // indirect | ||
| 53 | github.com/subosito/gotenv v1.6.0 // indirect | 53 | github.com/subosito/gotenv v1.6.0 // indirect | ||
M internal/infrastructure/webkit/context.go +10 -0
| @@ -94,6 +94,16 @@ func (c *WebKitContext) initNetworkSession() error { | |||
| 94 | 94 | return fmt.Errorf("website data manager is ephemeral") | |
| 95 | 95 | } | |
| 96 | 96 | ||
| 97 | + | // Debug: verify the directories WebKit is actually using | |
| 98 | + | actualDataDir := dataManager.GetBaseDataDirectory() | |
| 99 | + | actualCacheDir := dataManager.GetBaseCacheDirectory() | |
| 100 | + | c.logger.Info(). | |
| 101 | + | Str("expected_data_dir", c.dataDir). | |
| 102 | + | Str("actual_data_dir", actualDataDir). | |
| 103 | + | Str("expected_cache_dir", c.cacheDir). | |
| 104 | + | Str("actual_cache_dir", actualCacheDir). | |
| 105 | + | Msg("webkit directories configured") | |
| 106 | + | ||
| 97 | 107 | // Enable favicon collection and get database reference | |
| 98 | 108 | dataManager.SetFaviconsEnabled(true) | |
| 99 | 109 | c.faviconDB = dataManager.GetFaviconDatabase() | |
| @@ -94,6 +94,16 @@ func (c *WebKitContext) initNetworkSession() error { | |||||
| 94 | return fmt.Errorf("website data manager is ephemeral") | 94 | return fmt.Errorf("website data manager is ephemeral") | ||
| 95 | } | 95 | } | ||
| 96 | 96 | ||||
| 97 | + | // Debug: verify the directories WebKit is actually using | |||
| 98 | + | actualDataDir := dataManager.GetBaseDataDirectory() | |||
| 99 | + | actualCacheDir := dataManager.GetBaseCacheDirectory() | |||
| 100 | + | c.logger.Info(). | |||
| 101 | + | Str("expected_data_dir", c.dataDir). | |||
| 102 | + | Str("actual_data_dir", actualDataDir). | |||
| 103 | + | Str("expected_cache_dir", c.cacheDir). | |||
| 104 | + | Str("actual_cache_dir", actualCacheDir). | |||
| 105 | + | Msg("webkit directories configured") | |||
| 106 | + | ||||
| 97 | // Enable favicon collection and get database reference | 107 | // Enable favicon collection and get database reference | ||
| 98 | dataManager.SetFaviconsEnabled(true) | 108 | dataManager.SetFaviconsEnabled(true) | ||
| 99 | c.faviconDB = dataManager.GetFaviconDatabase() | 109 | c.faviconDB = dataManager.GetFaviconDatabase() | ||