Skip to content

Commit 487ef68

Browse files
leogdionclaude
andcommitted
fix: add missing jsonOutput parameter to SyncConfiguration
The configuration loader was not reading the BUSHEL_SYNC_JSON_OUTPUT environment variable, even though it was set in the workflow. The jsonOutput parameter was defined in ConfigurationKeys and SyncConfiguration, but was missing from the loader initialization. This caused the binary to output human-readable text instead of JSON, which corrupted the sync-result.json file and caused jq parsing errors. Root cause analysis: 1. Workflow set BUSHEL_SYNC_JSON_OUTPUT=true ✓ 2. ConfigurationKeys.Sync.jsonOutput defined ✓ 3. SyncConfiguration.jsonOutput parameter defined ✓ 4. ConfigurationLoader reading jsonOutput ✗ (MISSING - now fixed) 5. SyncCommand checking config.sync?.jsonOutput ✓ 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude Sonnet 4.5 <noreply@anthropic.com>
1 parent f54d412 commit 487ef68

File tree

1 file changed

+2
-1
lines changed

1 file changed

+2
-1
lines changed

Sources/BushelCloudKit/Configuration/ConfigurationLoader+Loading.swift

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -94,7 +94,8 @@ extension ConfigurationLoader {
9494
verbose: read(ConfigurationKeys.Sync.verbose),
9595
force: read(ConfigurationKeys.Sync.force),
9696
minInterval: read(ConfigurationKeys.Sync.minInterval),
97-
source: read(ConfigurationKeys.Sync.source)
97+
source: read(ConfigurationKeys.Sync.source),
98+
jsonOutput: read(ConfigurationKeys.Sync.jsonOutput)
9899
)
99100

100101
// Export command configuration

0 commit comments

Comments
 (0)