@quonfig/cli 0.0.32 → 0.0.34

This diff represents the content of publicly available package versions that have been released to one of the supported registries. The information contained in this diff is provided for informational purposes only and reflects changes between package versions as they appear in their respective public registries.
package/README.md CHANGED
@@ -14,7 +14,7 @@ $ npm install -g @quonfig/cli
14
14
  $ qfg COMMAND
15
15
  running command...
16
16
  $ qfg (--version)
17
- @quonfig/cli/0.0.32 linux-x64 node-v24.4.1
17
+ @quonfig/cli/0.0.34 linux-x64 node-v24.4.1
18
18
  $ qfg --help [COMMAND]
19
19
  USAGE
20
20
  $ qfg COMMAND
@@ -91,7 +91,7 @@ EXAMPLES
91
91
  $ qfg config-schema --json-schema # full JSON Schema document (copy into your editor)
92
92
  ```
93
93
 
94
- _See code: [src/commands/config-schema.ts](https://github.com/quonfig/cli/blob/v0.0.32/src/commands/config-schema.ts)_
94
+ _See code: [src/commands/config-schema.ts](https://github.com/quonfig/cli/blob/v0.0.34/src/commands/config-schema.ts)_
95
95
 
96
96
  ## `qfg create NAME`
97
97
 
@@ -176,7 +176,7 @@ EXAMPLES
176
176
  $ qfg set-rollout my.new.flag --environment production --true-percent 20
177
177
  ```
178
178
 
179
- _See code: [src/commands/create.ts](https://github.com/quonfig/cli/blob/v0.0.32/src/commands/create.ts)_
179
+ _See code: [src/commands/create.ts](https://github.com/quonfig/cli/blob/v0.0.34/src/commands/create.ts)_
180
180
 
181
181
  ## `qfg delete NAME`
182
182
 
@@ -222,7 +222,7 @@ EXAMPLES
222
222
  $ qfg delete my.config --yes
223
223
  ```
224
224
 
225
- _See code: [src/commands/delete.ts](https://github.com/quonfig/cli/blob/v0.0.32/src/commands/delete.ts)_
225
+ _See code: [src/commands/delete.ts](https://github.com/quonfig/cli/blob/v0.0.34/src/commands/delete.ts)_
226
226
 
227
227
  ## `qfg flag info [NAME]`
228
228
 
@@ -425,7 +425,7 @@ EXAMPLES
425
425
  $ qfg generate --targets node-ts -o ./dist # combine with targets
426
426
  ```
427
427
 
428
- _See code: [src/commands/generate.ts](https://github.com/quonfig/cli/blob/v0.0.32/src/commands/generate.ts)_
428
+ _See code: [src/commands/generate.ts](https://github.com/quonfig/cli/blob/v0.0.34/src/commands/generate.ts)_
429
429
 
430
430
  ## `qfg generate-new-hex-key`
431
431
 
@@ -448,7 +448,7 @@ EXAMPLES
448
448
  $ qfg generate-new-hex-key
449
449
  ```
450
450
 
451
- _See code: [src/commands/generate-new-hex-key.ts](https://github.com/quonfig/cli/blob/v0.0.32/src/commands/generate-new-hex-key.ts)_
451
+ _See code: [src/commands/generate-new-hex-key.ts](https://github.com/quonfig/cli/blob/v0.0.34/src/commands/generate-new-hex-key.ts)_
452
452
 
453
453
  ## `qfg get [NAME]`
454
454
 
@@ -480,7 +480,7 @@ EXAMPLES
480
480
  $ qfg get my.config.name --environment=production
481
481
  ```
482
482
 
483
- _See code: [src/commands/get.ts](https://github.com/quonfig/cli/blob/v0.0.32/src/commands/get.ts)_
483
+ _See code: [src/commands/get.ts](https://github.com/quonfig/cli/blob/v0.0.34/src/commands/get.ts)_
484
484
 
485
485
  ## `qfg info [NAME]`
486
486
 
@@ -528,7 +528,7 @@ EXAMPLES
528
528
  $ qfg info my.config.name --exclude-evaluations # skip 24h stats
529
529
  ```
530
530
 
531
- _See code: [src/commands/info.ts](https://github.com/quonfig/cli/blob/v0.0.32/src/commands/info.ts)_
531
+ _See code: [src/commands/info.ts](https://github.com/quonfig/cli/blob/v0.0.34/src/commands/info.ts)_
532
532
 
533
533
  ## `qfg init [DIRECTORY]`
534
534
 
@@ -568,7 +568,7 @@ EXAMPLES
568
568
  $ qfg init --dry-run
569
569
  ```
570
570
 
571
- _See code: [src/commands/init.ts](https://github.com/quonfig/cli/blob/v0.0.32/src/commands/init.ts)_
571
+ _See code: [src/commands/init.ts](https://github.com/quonfig/cli/blob/v0.0.34/src/commands/init.ts)_
572
572
 
573
573
  ## `qfg interactive`
574
574
 
@@ -598,7 +598,7 @@ EXAMPLES
598
598
  $ qfg
599
599
  ```
600
600
 
601
- _See code: [src/commands/interactive.ts](https://github.com/quonfig/cli/blob/v0.0.32/src/commands/interactive.ts)_
601
+ _See code: [src/commands/interactive.ts](https://github.com/quonfig/cli/blob/v0.0.34/src/commands/interactive.ts)_
602
602
 
603
603
  ## `qfg list`
604
604
 
@@ -638,7 +638,7 @@ EXAMPLES
638
638
  $ qfg list --feature-flags
639
639
  ```
640
640
 
641
- _See code: [src/commands/list.ts](https://github.com/quonfig/cli/blob/v0.0.32/src/commands/list.ts)_
641
+ _See code: [src/commands/list.ts](https://github.com/quonfig/cli/blob/v0.0.34/src/commands/list.ts)_
642
642
 
643
643
  ## `qfg log-level NAME`
644
644
 
@@ -703,7 +703,7 @@ EXAMPLES
703
703
  $ qfg log-level log-level.my-app --target=Chatty --value=INFO --environment=production
704
704
  ```
705
705
 
706
- _See code: [src/commands/log-level.ts](https://github.com/quonfig/cli/blob/v0.0.32/src/commands/log-level.ts)_
706
+ _See code: [src/commands/log-level.ts](https://github.com/quonfig/cli/blob/v0.0.34/src/commands/log-level.ts)_
707
707
 
708
708
  ## `qfg login`
709
709
 
@@ -728,7 +728,7 @@ EXAMPLES
728
728
  $ qfg login --profile myprofile
729
729
  ```
730
730
 
731
- _See code: [src/commands/login.ts](https://github.com/quonfig/cli/blob/v0.0.32/src/commands/login.ts)_
731
+ _See code: [src/commands/login.ts](https://github.com/quonfig/cli/blob/v0.0.34/src/commands/login.ts)_
732
732
 
733
733
  ## `qfg logout`
734
734
 
@@ -751,7 +751,7 @@ EXAMPLES
751
751
  $ qfg logout
752
752
  ```
753
753
 
754
- _See code: [src/commands/logout.ts](https://github.com/quonfig/cli/blob/v0.0.32/src/commands/logout.ts)_
754
+ _See code: [src/commands/logout.ts](https://github.com/quonfig/cli/blob/v0.0.34/src/commands/logout.ts)_
755
755
 
756
756
  ## `qfg migrate`
757
757
 
@@ -805,7 +805,7 @@ EXAMPLES
805
805
  $ qfg migrate --from launch --api-key $LAUNCH_API_KEY --dir ./quonfig-repo --staging
806
806
  ```
807
807
 
808
- _See code: [src/commands/migrate.ts](https://github.com/quonfig/cli/blob/v0.0.32/src/commands/migrate.ts)_
808
+ _See code: [src/commands/migrate.ts](https://github.com/quonfig/cli/blob/v0.0.34/src/commands/migrate.ts)_
809
809
 
810
810
  ## `qfg migrate doctor`
811
811
 
@@ -843,7 +843,7 @@ EXAMPLES
843
843
  $ qfg migrate doctor --json
844
844
  ```
845
845
 
846
- _See code: [src/commands/migrate/doctor.ts](https://github.com/quonfig/cli/blob/v0.0.32/src/commands/migrate/doctor.ts)_
846
+ _See code: [src/commands/migrate/doctor.ts](https://github.com/quonfig/cli/blob/v0.0.34/src/commands/migrate/doctor.ts)_
847
847
 
848
848
  ## `qfg migrate my-code`
849
849
 
@@ -875,7 +875,7 @@ EXAMPLES
875
875
  $ qfg migrate my-code --dry-run
876
876
  ```
877
877
 
878
- _See code: [src/commands/migrate/my-code.ts](https://github.com/quonfig/cli/blob/v0.0.32/src/commands/migrate/my-code.ts)_
878
+ _See code: [src/commands/migrate/my-code.ts](https://github.com/quonfig/cli/blob/v0.0.34/src/commands/migrate/my-code.ts)_
879
879
 
880
880
  ## `qfg migrate status`
881
881
 
@@ -906,7 +906,7 @@ EXAMPLES
906
906
  $ qfg migrate status --json
907
907
  ```
908
908
 
909
- _See code: [src/commands/migrate/status.ts](https://github.com/quonfig/cli/blob/v0.0.32/src/commands/migrate/status.ts)_
909
+ _See code: [src/commands/migrate/status.ts](https://github.com/quonfig/cli/blob/v0.0.34/src/commands/migrate/status.ts)_
910
910
 
911
911
  ## `qfg override [NAME] [VALUE]`
912
912
 
@@ -961,7 +961,7 @@ EXAMPLES
961
961
  $ qfg override my.flag true --env=staging
962
962
  ```
963
963
 
964
- _See code: [src/commands/override.ts](https://github.com/quonfig/cli/blob/v0.0.32/src/commands/override.ts)_
964
+ _See code: [src/commands/override.ts](https://github.com/quonfig/cli/blob/v0.0.34/src/commands/override.ts)_
965
965
 
966
966
  ## `qfg pull`
967
967
 
@@ -969,10 +969,13 @@ Clone or update a local copy of your workspace config files.
969
969
 
970
970
  ```
971
971
  USAGE
972
- $ qfg pull [--json] [--interactive] [--no-color] [--verbose] [--dir <value>] [--workspace <value>]
972
+ $ qfg pull [--json] [--interactive] [--no-color] [--verbose] [--dir <value>] [--rebase] [--workspace
973
+ <value>]
973
974
 
974
975
  FLAGS
975
976
  --dir=<value> Local directory to clone/update (defaults to QUONFIG_DIR env var)
977
+ --rebase Replay local commits on top of origin/main when the two have diverged. Conflicts are surfaced via
978
+ standard git markers; resolve with `git rebase --continue`.
976
979
  --workspace=<value> Workspace ID (defaults to active profile)
977
980
 
978
981
  GLOBAL FLAGS
@@ -1005,7 +1008,7 @@ EXAMPLES
1005
1008
  $ qfg pull # uses QUONFIG_DIR env var
1006
1009
  ```
1007
1010
 
1008
- _See code: [src/commands/pull.ts](https://github.com/quonfig/cli/blob/v0.0.32/src/commands/pull.ts)_
1011
+ _See code: [src/commands/pull.ts](https://github.com/quonfig/cli/blob/v0.0.34/src/commands/pull.ts)_
1009
1012
 
1010
1013
  ## `qfg push`
1011
1014
 
@@ -1053,7 +1056,7 @@ EXAMPLES
1053
1056
  $ qfg push --dir ./our-config --yes
1054
1057
  ```
1055
1058
 
1056
- _See code: [src/commands/push.ts](https://github.com/quonfig/cli/blob/v0.0.32/src/commands/push.ts)_
1059
+ _See code: [src/commands/push.ts](https://github.com/quonfig/cli/blob/v0.0.34/src/commands/push.ts)_
1057
1060
 
1058
1061
  ## `qfg schema NAME`
1059
1062
 
@@ -1091,7 +1094,7 @@ EXAMPLES
1091
1094
  $ qfg schema my-schema --set-json-schema=@schemas/my-schema.json --protected
1092
1095
  ```
1093
1096
 
1094
- _See code: [src/commands/schema.ts](https://github.com/quonfig/cli/blob/v0.0.32/src/commands/schema.ts)_
1097
+ _See code: [src/commands/schema.ts](https://github.com/quonfig/cli/blob/v0.0.34/src/commands/schema.ts)_
1095
1098
 
1096
1099
  ## `qfg sdk-key`
1097
1100
 
@@ -1118,7 +1121,7 @@ EXAMPLES
1118
1121
  $ qfg sdk-key revoke <key-id>
1119
1122
  ```
1120
1123
 
1121
- _See code: [src/commands/sdk-key.ts](https://github.com/quonfig/cli/blob/v0.0.32/src/commands/sdk-key.ts)_
1124
+ _See code: [src/commands/sdk-key.ts](https://github.com/quonfig/cli/blob/v0.0.34/src/commands/sdk-key.ts)_
1122
1125
 
1123
1126
  ## `qfg sdk-key create`
1124
1127
 
@@ -1150,7 +1153,7 @@ EXAMPLES
1150
1153
  $ qfg sdk-key create --environment staging --type browser
1151
1154
  ```
1152
1155
 
1153
- _See code: [src/commands/sdk-key/create.ts](https://github.com/quonfig/cli/blob/v0.0.32/src/commands/sdk-key/create.ts)_
1156
+ _See code: [src/commands/sdk-key/create.ts](https://github.com/quonfig/cli/blob/v0.0.34/src/commands/sdk-key/create.ts)_
1154
1157
 
1155
1158
  ## `qfg sdk-key list`
1156
1159
 
@@ -1179,7 +1182,7 @@ EXAMPLES
1179
1182
  $ qfg sdk-key list --environment production
1180
1183
  ```
1181
1184
 
1182
- _See code: [src/commands/sdk-key/list.ts](https://github.com/quonfig/cli/blob/v0.0.32/src/commands/sdk-key/list.ts)_
1185
+ _See code: [src/commands/sdk-key/list.ts](https://github.com/quonfig/cli/blob/v0.0.34/src/commands/sdk-key/list.ts)_
1183
1186
 
1184
1187
  ## `qfg sdk-key revoke KEYID`
1185
1188
 
@@ -1206,7 +1209,7 @@ EXAMPLES
1206
1209
  $ qfg sdk-key revoke a1b2c3d4-e5f6-7890-abcd-ef1234567890
1207
1210
  ```
1208
1211
 
1209
- _See code: [src/commands/sdk-key/revoke.ts](https://github.com/quonfig/cli/blob/v0.0.32/src/commands/sdk-key/revoke.ts)_
1212
+ _See code: [src/commands/sdk-key/revoke.ts](https://github.com/quonfig/cli/blob/v0.0.34/src/commands/sdk-key/revoke.ts)_
1210
1213
 
1211
1214
  ## `qfg set-default [NAME]`
1212
1215
 
@@ -1277,7 +1280,7 @@ EXAMPLES
1277
1280
  $ qfg pull && qfg config-schema
1278
1281
  ```
1279
1282
 
1280
- _See code: [src/commands/set-default.ts](https://github.com/quonfig/cli/blob/v0.0.32/src/commands/set-default.ts)_
1283
+ _See code: [src/commands/set-default.ts](https://github.com/quonfig/cli/blob/v0.0.34/src/commands/set-default.ts)_
1281
1284
 
1282
1285
  ## `qfg set-rollout [NAME]`
1283
1286
 
@@ -1354,7 +1357,7 @@ EXAMPLES
1354
1357
  $ qfg set-rollout my.variant.flag --environment production --weights "a:33,b:33,c:34" --hash-by user.id
1355
1358
  ```
1356
1359
 
1357
- _See code: [src/commands/set-rollout.ts](https://github.com/quonfig/cli/blob/v0.0.32/src/commands/set-rollout.ts)_
1360
+ _See code: [src/commands/set-rollout.ts](https://github.com/quonfig/cli/blob/v0.0.34/src/commands/set-rollout.ts)_
1358
1361
 
1359
1362
  ## `qfg sync`
1360
1363
 
@@ -1386,7 +1389,7 @@ EXAMPLES
1386
1389
  $ qfg sync --watch --dir ./our-config --interval 30
1387
1390
  ```
1388
1391
 
1389
- _See code: [src/commands/sync.ts](https://github.com/quonfig/cli/blob/v0.0.32/src/commands/sync.ts)_
1392
+ _See code: [src/commands/sync.ts](https://github.com/quonfig/cli/blob/v0.0.34/src/commands/sync.ts)_
1390
1393
 
1391
1394
  ## `qfg toggle [NAME]`
1392
1395
 
@@ -1488,7 +1491,7 @@ EXAMPLES
1488
1491
  $ qfg verify --json
1489
1492
  ```
1490
1493
 
1491
- _See code: [src/commands/verify.ts](https://github.com/quonfig/cli/blob/v0.0.32/src/commands/verify.ts)_
1494
+ _See code: [src/commands/verify.ts](https://github.com/quonfig/cli/blob/v0.0.34/src/commands/verify.ts)_
1492
1495
 
1493
1496
  ## `qfg whoami`
1494
1497
 
@@ -1511,7 +1514,7 @@ EXAMPLES
1511
1514
  $ qfg whoami
1512
1515
  ```
1513
1516
 
1514
- _See code: [src/commands/whoami.ts](https://github.com/quonfig/cli/blob/v0.0.32/src/commands/whoami.ts)_
1517
+ _See code: [src/commands/whoami.ts](https://github.com/quonfig/cli/blob/v0.0.34/src/commands/whoami.ts)_
1515
1518
 
1516
1519
  ## `qfg workspace`
1517
1520
 
@@ -1534,7 +1537,7 @@ EXAMPLES
1534
1537
  $ qfg workspace
1535
1538
  ```
1536
1539
 
1537
- _See code: [src/commands/workspace.ts](https://github.com/quonfig/cli/blob/v0.0.32/src/commands/workspace.ts)_
1540
+ _See code: [src/commands/workspace.ts](https://github.com/quonfig/cli/blob/v0.0.34/src/commands/workspace.ts)_
1538
1541
 
1539
1542
  ## `qfg workspace bootstrap`
1540
1543
 
@@ -1567,7 +1570,7 @@ EXAMPLES
1567
1570
  $ qfg workspace bootstrap --dir ./our-config --skip-validate
1568
1571
  ```
1569
1572
 
1570
- _See code: [src/commands/workspace/bootstrap.ts](https://github.com/quonfig/cli/blob/v0.0.32/src/commands/workspace/bootstrap.ts)_
1573
+ _See code: [src/commands/workspace/bootstrap.ts](https://github.com/quonfig/cli/blob/v0.0.34/src/commands/workspace/bootstrap.ts)_
1571
1574
 
1572
1575
  ## `qfg workspace create SLUG`
1573
1576
 
@@ -1601,7 +1604,7 @@ EXAMPLES
1601
1604
  $ qfg workspace create my-team --org 11111111-1111-1111-1111-111111111111
1602
1605
  ```
1603
1606
 
1604
- _See code: [src/commands/workspace/create.ts](https://github.com/quonfig/cli/blob/v0.0.32/src/commands/workspace/create.ts)_
1607
+ _See code: [src/commands/workspace/create.ts](https://github.com/quonfig/cli/blob/v0.0.34/src/commands/workspace/create.ts)_
1605
1608
 
1606
1609
  ## `qfg workspace switch [SLUG]`
1607
1610
 
@@ -1629,7 +1632,7 @@ EXAMPLES
1629
1632
  $ qfg workspace switch acme/production
1630
1633
  ```
1631
1634
 
1632
- _See code: [src/commands/workspace/switch.ts](https://github.com/quonfig/cli/blob/v0.0.32/src/commands/workspace/switch.ts)_
1635
+ _See code: [src/commands/workspace/switch.ts](https://github.com/quonfig/cli/blob/v0.0.34/src/commands/workspace/switch.ts)_
1633
1636
  <!-- commandsstop -->
1634
1637
 
1635
1638
  ## Local Development
@@ -4,8 +4,9 @@ import path from 'node:path';
4
4
  import { BaseCommand } from '../index.js';
5
5
  import { CrossSourceError, assertSourceMatches, readImportState } from '../migrate/import-state.js';
6
6
  import { applyLocalMigration } from '../migrate/local-write.js';
7
- import { PushConflictError } from '../util/clone-and-stack-push.js';
8
- import { pushMigrationToCloud } from '../migrate/push-to-cloud.js';
7
+ import { buildPushConflictSuggestion } from '../migrate/migrate-suggestion.js';
8
+ import { PushConflictError, PushHookRejectedError } from '../util/clone-and-stack-push.js';
9
+ import { MigratorVerifyError, pushMigrationToCloud } from '../migrate/push-to-cloud.js';
9
10
  import { UnknownSourceError, getSource } from '../migrate/registry.js';
10
11
  import { NotYetImplementedError, } from '../migrate/source.js';
11
12
  import { applyLaunchBaseUrl } from '../migrate/sources/launch/api.js';
@@ -171,8 +172,16 @@ export default class Migrate extends BaseCommand {
171
172
  };
172
173
  }
173
174
  catch (error) {
175
+ if (error instanceof PushHookRejectedError) {
176
+ return this.err(`${error.message}\n\nThe remote validation hook rejected the push. Fix the errors reported above (typically in the source system or by editing the local files in ${dir}), then re-run with --push. You can re-run \`qfg verify ${dir}\` locally to confirm the fixes before pushing.`);
177
+ }
174
178
  if (error instanceof PushConflictError) {
175
- return this.err(`${error.message}\n\nRe-run \`qfg migrate --from ${flags.from} --workspace ${workspaceId} --push\` to pick up remote changes before retrying.`);
179
+ return this.err(`${error.message}\n\n${buildPushConflictSuggestion({ from: flags.from, userWorkspaceFlag: flags.workspace })}`);
180
+ }
181
+ if (error instanceof MigratorVerifyError) {
182
+ return this.err(`${error.message}\n\nThe migrated workspace was written to ${dir} but was NOT pushed. ` +
183
+ `Fix the issues above (typically in the source system or by editing the local files), then re-run with --push. ` +
184
+ `You can re-run \`qfg verify ${dir}\` locally to confirm the fixes before pushing.`);
176
185
  }
177
186
  throw error;
178
187
  }
@@ -213,6 +222,7 @@ export default class Migrate extends BaseCommand {
213
222
  // detectDuplicateKeys). The source accumulator holds the data regardless.
214
223
  warnAboutDroppedOverrides(this, source.getDroppedOverrides?.() ?? null);
215
224
  warnAboutSkippedConfigs(this, source.getSkippedConfigs?.() ?? null);
225
+ warnAboutCoercedSentinels(this, source.getCoercedSentinels?.() ?? null);
216
226
  warnAboutDuplicateResolutions(this, duplicateResolutionsForWarn);
217
227
  }
218
228
  }
@@ -283,6 +293,21 @@ function warnAboutDuplicateResolutions(cmd, resolved) {
283
293
  }
284
294
  cmd.warn('Full detail is also written to MIGRATION_REPORT.md.');
285
295
  }
296
+ function warnAboutCoercedSentinels(cmd, coerced) {
297
+ if (!coerced || coerced.total === 0)
298
+ return;
299
+ const envIds = Object.keys(coerced.byEnv).sort();
300
+ cmd.warn(`Coerced ${coerced.total} sentinel rule value(s) from Launch's "no value set yet" sentinel ({type:"string", value:""}) to the typed default for the surrounding config. Without coercion the qfg-verify hook would reject these as type-mismatches and fail-stop the entire push:`);
301
+ for (const envId of envIds) {
302
+ const perFlag = coerced.byEnv[envId];
303
+ const totalForEnv = Object.values(perFlag).reduce((s, n) => s + n, 0);
304
+ cmd.warn(` env-${envId}: ${totalForEnv} coerced from ${Object.keys(perFlag).length} config(s)`);
305
+ for (const flagPath of Object.keys(perFlag).sort()) {
306
+ cmd.warn(` - ${flagPath} (${perFlag[flagPath]})`);
307
+ }
308
+ }
309
+ cmd.warn(`If you want a real default for these rules, set one in the source system and re-run the migration. Full detail is also written to MIGRATION_REPORT.md.`);
310
+ }
286
311
  function warnAboutSkippedConfigs(cmd, skipped) {
287
312
  if (!skipped || skipped.total === 0)
288
313
  return;
@@ -5,6 +5,7 @@ export default class Pull extends BaseCommand {
5
5
  static examples: string[];
6
6
  static flags: {
7
7
  dir: import("@oclif/core/lib/interfaces/parser.js").OptionFlag<string | undefined, import("@oclif/core/lib/interfaces/parser.js").CustomOptions>;
8
+ rebase: import("@oclif/core/lib/interfaces/parser.js").BooleanFlag<boolean>;
8
9
  workspace: import("@oclif/core/lib/interfaces/parser.js").OptionFlag<string | undefined, import("@oclif/core/lib/interfaces/parser.js").CustomOptions>;
9
10
  };
10
11
  run(): Promise<JsonObj | void>;
@@ -6,7 +6,7 @@ import { loadGiteaToken, isGiteaTokenExpired, saveGiteaToken } from '../util/git
6
6
  import { mintAndStoreGiteaReadToken, mintGiteaToken } from '../util/gitea-api.js';
7
7
  import { readWorkspaceSlug, tryParseWorkspacePin, writeWorkspaceSlug } from '../util/quonfig-json.js';
8
8
  import { resolveWorkspaceUuid } from '../util/resolve-workspace.js';
9
- import { isGitRepo, getRemoteUrl, gitFetch, gitSetRemote, canFastForward, hasDivergedFromRemote, gitMergeFfOnly, gitClone, displayUrl, addAndCommitFile, dirtyTrackedFiles, } from '../util/git-ops.js';
9
+ import { isGitRepo, getRemoteUrl, gitFetch, gitSetRemote, canFastForward, hasDivergedFromRemote, gitMergeFfOnly, gitPullRebase, gitClone, displayUrl, addAndCommitFile, dirtyTrackedFiles, } from '../util/git-ops.js';
10
10
  export default class Pull extends BaseCommand {
11
11
  static description = `Clone or update a local copy of your workspace config files.
12
12
 
@@ -29,6 +29,10 @@ CLI shortcuts (no JSON editing needed for simple cases):
29
29
  dir: Flags.string({
30
30
  description: 'Local directory to clone/update (defaults to QUONFIG_DIR env var)',
31
31
  }),
32
+ rebase: Flags.boolean({
33
+ default: false,
34
+ description: 'Replay local commits on top of origin/main when the two have diverged. Conflicts are surfaced via standard git markers; resolve with `git rebase --continue`.',
35
+ }),
32
36
  workspace: Flags.string({
33
37
  description: 'Workspace ID (defaults to active profile)',
34
38
  }),
@@ -85,7 +89,43 @@ CLI shortcuts (no JSON editing needed for simple cases):
85
89
  const ff = await canFastForward(resolvedDir);
86
90
  const diverged = !ff && (await hasDivergedFromRemote(resolvedDir));
87
91
  if (diverged) {
88
- return this.err('Local commits diverge from remote — resolve manually.');
92
+ if (flags.rebase) {
93
+ this.log('Local and origin/main have diverged. Rebasing local commits on top of origin/main...');
94
+ const result = await gitPullRebase(resolvedDir);
95
+ if (result.kind === 'clean') {
96
+ this.log(`Rebased ${result.commitsRebased} local commit(s) onto origin/main. Run \`qfg push\` to publish.`);
97
+ // Fall through to post-pull work (QUONFIG_DIR write, pin backfill).
98
+ }
99
+ else if (result.kind === 'conflicts') {
100
+ this.log('');
101
+ this.log('Rebase paused with conflicts in:');
102
+ for (const f of result.conflictedFiles)
103
+ this.log(` ${f}`);
104
+ this.log('');
105
+ this.log('To finish the rebase:');
106
+ this.log(` 1. Edit each file above and pick the values you want; remove the git conflict markers.`);
107
+ this.log(` 2. git -C ${resolvedDir} add <files>`);
108
+ this.log(` 3. git -C ${resolvedDir} rebase --continue`);
109
+ this.log(` 4. qfg push`);
110
+ this.log('');
111
+ this.log('To abort the rebase and try a different approach:');
112
+ this.log(` git -C ${resolvedDir} rebase --abort`);
113
+ return this.err('Rebase paused on conflicts — see instructions above.');
114
+ }
115
+ else {
116
+ return this.err(`Rebase could not start: ${result.reason}\n\n` +
117
+ `Most likely the working tree has uncommitted edits. Commit or stash them, then re-run \`qfg pull --rebase\`.`);
118
+ }
119
+ }
120
+ else {
121
+ return this.err(`Local commits diverge from origin/main.\n\n` +
122
+ `Your local branch has commits that are not on origin, and origin has commits that are not local. ` +
123
+ `qfg pull will not merge automatically. Pick one:\n\n` +
124
+ ` Replay your local commits on top of origin/main:\n` +
125
+ ` qfg pull --rebase --dir ${resolvedDir}\n\n` +
126
+ ` Discard your local commits and take origin/main as-is (LOSES local work):\n` +
127
+ ` git -C ${resolvedDir} reset --hard origin/main`);
128
+ }
89
129
  }
90
130
  if (ff) {
91
131
  const newCommits = await gitMergeFfOnly(resolvedDir);
@@ -3,7 +3,7 @@ import * as path from 'node:path';
3
3
  import { Flags } from '@oclif/core';
4
4
  import { BaseCommand } from '../index.js';
5
5
  import { mintGiteaToken } from '../util/gitea-api.js';
6
- import { getRemoteUrl, gitFetch, gitSetRemote, isGitRepo, runGit } from '../util/git-ops.js';
6
+ import { dirtyTrackedFiles, getRemoteUrl, gitFetch, gitSetRemote, isGitRepo, isLocalBehindOrDivergedFromRemote, runGit } from '../util/git-ops.js';
7
7
  import { PushFatalError, runPush, } from '../push/run-push.js';
8
8
  import { computeBarePathDiff } from '../push/bare-path-diff.js';
9
9
  import { callConfigsPush } from '../push/config-push-client.js';
@@ -202,6 +202,29 @@ export function buildRealDeps(cmd, orgSlug) {
202
202
  const probe = await ensureBarePathProbe(dir);
203
203
  return probe.totalRemoteFiles;
204
204
  },
205
+ async isLocalBehindRemote(dir) {
206
+ // qfg-fboj: stale-HEAD guard for the clone path. Implementation
207
+ // lives in git-ops.ts so it can be unit-tested against a real
208
+ // local git repo. We still gate on isGitRepo here because the
209
+ // helper does not (it returns false on any error, but we want to
210
+ // skip the rev-parse cost on the bare path entirely).
211
+ if (!(await isGitRepo(dir)))
212
+ return false;
213
+ return isLocalBehindOrDivergedFromRemote(dir);
214
+ },
215
+ async dirtyTrackedFiles(dir) {
216
+ // qfg-fboj: surface uncommitted edits so the user sees they were
217
+ // not included in this push. Only meaningful on the clone path —
218
+ // bare path doesn't have a .git/ to status.
219
+ if (!(await isGitRepo(dir)))
220
+ return [];
221
+ try {
222
+ return await dirtyTrackedFiles(dir);
223
+ }
224
+ catch {
225
+ return [];
226
+ }
227
+ },
205
228
  };
206
229
  const deps = {
207
230
  async mintWriteToken(requestedTarget) {
@@ -1,7 +1,7 @@
1
1
  import { type ImportState } from './import-state.js';
2
2
  import { type MigrationReportData } from './migration-report.js';
3
3
  import { type PushIdentity } from '../util/clone-and-stack-push.js';
4
- import type { DroppedOverrideSummary, DuplicateResolutionSummary, LegacyChange, MigrationSource, SkippedConfigSummary } from './source.js';
4
+ import type { CoercedSentinelSummary, DroppedOverrideSummary, DuplicateResolutionSummary, LegacyChange, MigrationSource, SkippedConfigSummary } from './source.js';
5
5
  export interface ApplyLocalMigrationOptions {
6
6
  /** Commit author. Defaults to the migrator identity. */
7
7
  author?: PushIdentity;
@@ -19,6 +19,8 @@ export interface ApplyLocalMigrationOptions {
19
19
  export interface ApplyLocalMigrationResult {
20
20
  /** `'initialized'` if we had to `git init` the target, `'reused'` if it was already a repo. */
21
21
  action: 'initialized' | 'reused';
22
+ /** Sentinel rule values coerced to typed defaults during translate. Null if none. */
23
+ coercedSentinels: CoercedSentinelSummary | null;
22
24
  commitSha: string | null;
23
25
  /** `false` if translate produced no net changes — nothing was committed. */
24
26
  committed: boolean;
@@ -85,9 +85,11 @@ export const applyLocalMigration = async (opts) => {
85
85
  writeImportState(opts.localDir, opts.importState);
86
86
  const droppedOverrides = opts.source.getDroppedOverrides?.() ?? null;
87
87
  const skippedConfigs = opts.source.getSkippedConfigs?.() ?? null;
88
+ const coercedSentinels = opts.source.getCoercedSentinels?.() ?? null;
88
89
  const duplicateResolutions = resolutionEntries.length > 0 ? { entries: resolutionEntries, total: resolutionEntries.length } : null;
89
90
  const reportData = {
90
91
  ...opts.reportData,
92
+ ...(coercedSentinels ? { coercedSentinels } : {}),
91
93
  ...(droppedOverrides ? { droppedOverrides } : {}),
92
94
  ...(duplicateResolutions ? { duplicateResolutions } : {}),
93
95
  ...(skippedConfigs ? { skippedConfigs } : {}),
@@ -95,7 +97,15 @@ export const applyLocalMigration = async (opts) => {
95
97
  writeMigrationReport(opts.localDir, reportData);
96
98
  await runGit(['-C', opts.localDir, 'add', '--all']);
97
99
  if (!(await hasStagedChanges(opts.localDir))) {
98
- return { action, committed: false, commitSha: null, droppedOverrides, duplicateResolutions, skippedConfigs };
100
+ return {
101
+ action,
102
+ coercedSentinels,
103
+ committed: false,
104
+ commitSha: null,
105
+ droppedOverrides,
106
+ duplicateResolutions,
107
+ skippedConfigs,
108
+ };
99
109
  }
100
110
  await spawnGit(['-C', opts.localDir, 'commit', '-F', '-'], {
101
111
  stdin: opts.commitMessage,
@@ -109,6 +119,7 @@ export const applyLocalMigration = async (opts) => {
109
119
  const { stdout: sha } = await runGit(['-C', opts.localDir, 'rev-parse', 'HEAD']);
110
120
  return {
111
121
  action,
122
+ coercedSentinels,
112
123
  commitSha: sha.trim(),
113
124
  committed: true,
114
125
  droppedOverrides,
@@ -0,0 +1,12 @@
1
+ export interface PushConflictSuggestionInput {
2
+ from: string;
3
+ /**
4
+ * The original `--workspace` flag string the user typed, e.g.
5
+ * `test-organization/semgrep-test-1`. Pass `undefined` if the user did not
6
+ * pass `--workspace` (and the command resolved the workspace via the saved
7
+ * profile). Never pass the resolved workspace UUID — the suggestion is meant
8
+ * to be a copy-paste of the user's original invocation.
9
+ */
10
+ userWorkspaceFlag: string | undefined;
11
+ }
12
+ export declare function buildPushConflictSuggestion({ from, userWorkspaceFlag }: PushConflictSuggestionInput): string;
@@ -0,0 +1,4 @@
1
+ export function buildPushConflictSuggestion({ from, userWorkspaceFlag }) {
2
+ const workspaceArg = userWorkspaceFlag ? ` --workspace ${userWorkspaceFlag}` : '';
3
+ return `Re-run \`qfg migrate --from ${from}${workspaceArg} --push\` to pick up remote changes before retrying.`;
4
+ }
@@ -1,5 +1,5 @@
1
1
  import type { IdentifierMap } from './identifier-map.js';
2
- import type { DroppedOverrideSummary, DuplicateResolutionSummary, SkippedConfigSummary } from './source.js';
2
+ import type { CoercedSentinelSummary, DroppedOverrideSummary, DuplicateResolutionSummary, SkippedConfigSummary } from './source.js';
3
3
  export interface MigrationReportCounts {
4
4
  environmentsMapped: number;
5
5
  flagsMigrated: number;
@@ -29,6 +29,12 @@ export interface FollowUpChecklist {
29
29
  }
30
30
  export interface MigrationReportData {
31
31
  cleanMappings: CleanMapping[];
32
+ /**
33
+ * Rule values that translate() coerced from a sentinel like Launch's
34
+ * empty-string "no value set yet" to the typed default. Null when nothing
35
+ * was coerced.
36
+ */
37
+ coercedSentinels?: CoercedSentinelSummary | null;
32
38
  counts: MigrationReportCounts;
33
39
  /**
34
40
  * Override sections that were dropped during translate() because the env.id was not
@@ -87,6 +87,27 @@ const renderDuplicateResolutions = (resolved) => {
87
87
  }
88
88
  return lines.join('\n');
89
89
  };
90
+ const renderCoercedSentinels = (coerced) => {
91
+ if (!coerced || coerced.total === 0)
92
+ return null;
93
+ const envIds = Object.keys(coerced.byEnv).sort();
94
+ const lines = [
95
+ '## Coerced sentinel rule values',
96
+ '',
97
+ `Coerced **${coerced.total}** rule value(s) from Launch's "no value set yet" sentinel ({type:"string", value:""}) to the typed default for the surrounding config. The qfg-verify hook would otherwise reject these as type-mismatches and fail-stop the entire push. Affected envs: **${envIds.length}**. Review each and set a real default in the source system if needed.`,
98
+ ];
99
+ for (const envId of envIds) {
100
+ const perFlag = coerced.byEnv[envId];
101
+ const totalForEnv = Object.values(perFlag).reduce((s, n) => s + n, 0);
102
+ const flagCount = Object.keys(perFlag).length;
103
+ lines.push('', `### env-${envId} — ${totalForEnv} coerced from ${flagCount} config(s)`, '');
104
+ const sorted = Object.keys(perFlag).sort();
105
+ for (const flagPath of sorted) {
106
+ lines.push(`- \`${flagPath}\` (${perFlag[flagPath]})`);
107
+ }
108
+ }
109
+ return lines.join('\n');
110
+ };
90
111
  const renderSkippedConfigs = (skipped) => {
91
112
  if (!skipped || skipped.total === 0)
92
113
  return null;
@@ -136,6 +157,9 @@ export const buildMigrationReport = (data) => {
136
157
  const skipped = renderSkippedConfigs(data.skippedConfigs);
137
158
  if (skipped !== null)
138
159
  sections.push(skipped);
160
+ const coerced = renderCoercedSentinels(data.coercedSentinels);
161
+ if (coerced !== null)
162
+ sections.push(coerced);
139
163
  sections.push(renderFollowUp(data.followUp));
140
164
  return sections.join('\n\n') + '\n';
141
165
  };
@@ -1,7 +1,19 @@
1
1
  import { type ImportState } from './import-state.js';
2
2
  import { type MigrationReportData } from './migration-report.js';
3
3
  import { type CloneAndStackPushResult } from '../util/clone-and-stack-push.js';
4
- import type { DroppedOverrideSummary, DuplicateResolutionSummary, LegacyChange, MigrationSource, SkippedConfigSummary } from './source.js';
4
+ import { type ValidationResult } from '../verify/validate.js';
5
+ import type { CoercedSentinelSummary, DroppedOverrideSummary, DuplicateResolutionSummary, LegacyChange, MigrationSource, SkippedConfigSummary } from './source.js';
6
+ /**
7
+ * Thrown when the migrator's freshly-written workspace fails the same checks
8
+ * that the Gitea pre-receive `qfg-verify` hook will run server-side. We raise
9
+ * this client-side after `applyDelta` writes files but before commit + push,
10
+ * so the customer sees the failure locally rather than after a full push
11
+ * round-trip is rejected (qfg-52qg).
12
+ */
13
+ export declare class MigratorVerifyError extends Error {
14
+ readonly result: ValidationResult;
15
+ constructor(result: ValidationResult);
16
+ }
5
17
  export interface PushMigrationToCloudOptions {
6
18
  branch?: string;
7
19
  changes: LegacyChange[];
@@ -19,6 +31,7 @@ export interface PushMigrationToCloudOptions {
19
31
  source: MigrationSource;
20
32
  }
21
33
  export interface PushMigrationToCloudResult extends CloneAndStackPushResult {
34
+ coercedSentinels: CoercedSentinelSummary | null;
22
35
  droppedOverrides: DroppedOverrideSummary | null;
23
36
  duplicateResolutions: DuplicateResolutionSummary | null;
24
37
  skippedConfigs: SkippedConfigSummary | null;
@@ -4,6 +4,25 @@ import { removeQfFromGitignore, writeImportState } from './import-state.js';
4
4
  import { writeMigrationReport } from './migration-report.js';
5
5
  import { detectDuplicateKeys } from './sources/launch/translate.js';
6
6
  import { cloneAndStackPush, } from '../util/clone-and-stack-push.js';
7
+ import { formatResult, validateWorkspace } from '../verify/validate.js';
8
+ /**
9
+ * Thrown when the migrator's freshly-written workspace fails the same checks
10
+ * that the Gitea pre-receive `qfg-verify` hook will run server-side. We raise
11
+ * this client-side after `applyDelta` writes files but before commit + push,
12
+ * so the customer sees the failure locally rather than after a full push
13
+ * round-trip is rejected (qfg-52qg).
14
+ */
15
+ export class MigratorVerifyError extends Error {
16
+ result;
17
+ constructor(result) {
18
+ const errorCount = result.issues.filter((i) => i.severity === 'error').length;
19
+ super(`Migrator output failed qfg verify: ${errorCount} error(s) in ${result.filesChecked} file(s). ` +
20
+ `Refusing to push — the same errors would be rejected by the server pre-receive hook.\n\n` +
21
+ formatResult(result));
22
+ this.name = 'MigratorVerifyError';
23
+ this.result = result;
24
+ }
25
+ }
7
26
  const writeQuonfigFiles = (dir, changes, source) => {
8
27
  const livePaths = new Map();
9
28
  for (const change of changes) {
@@ -65,6 +84,7 @@ const mergeEnvironmentsIntoQuonfigJson = (dir, sourceEnvs) => {
65
84
  fs.writeFileSync(quonfigPath, JSON.stringify({ ...existing, environments: merged }, null, 2) + '\n', 'utf8');
66
85
  };
67
86
  export const pushMigrationToCloud = async (opts) => {
87
+ let coercedSentinels = null;
68
88
  let droppedOverrides = null;
69
89
  let duplicateResolutions = null;
70
90
  let skippedConfigs = null;
@@ -78,15 +98,24 @@ export const pushMigrationToCloud = async (opts) => {
78
98
  writeImportState(dir, opts.importState);
79
99
  droppedOverrides = opts.source.getDroppedOverrides?.() ?? null;
80
100
  skippedConfigs = opts.source.getSkippedConfigs?.() ?? null;
101
+ coercedSentinels = opts.source.getCoercedSentinels?.() ?? null;
81
102
  duplicateResolutions =
82
103
  resolutionEntries.length > 0 ? { entries: resolutionEntries, total: resolutionEntries.length } : null;
83
104
  const reportData = {
84
105
  ...opts.reportData,
106
+ ...(coercedSentinels ? { coercedSentinels } : {}),
85
107
  ...(droppedOverrides ? { droppedOverrides } : {}),
86
108
  ...(duplicateResolutions ? { duplicateResolutions } : {}),
87
109
  ...(skippedConfigs ? { skippedConfigs } : {}),
88
110
  };
89
111
  writeMigrationReport(dir, reportData);
112
+ // Pre-flight: run the same validation the Gitea pre-receive `qfg-verify`
113
+ // hook will run after the push lands. Failing here saves a full
114
+ // git-push round-trip and gives the customer a local, fixable error
115
+ // (qfg-52qg).
116
+ const verifyResult = validateWorkspace(dir);
117
+ if (!verifyResult.valid)
118
+ throw new MigratorVerifyError(verifyResult);
90
119
  },
91
120
  commitMessage: opts.commitMessage,
92
121
  localDir: opts.localDir,
@@ -95,5 +124,5 @@ export const pushMigrationToCloud = async (opts) => {
95
124
  if (opts.branch !== undefined)
96
125
  cloneOpts.branch = opts.branch;
97
126
  const result = await cloneAndStackPush(cloneOpts);
98
- return { ...result, droppedOverrides, duplicateResolutions, skippedConfigs };
127
+ return { ...result, coercedSentinels, droppedOverrides, duplicateResolutions, skippedConfigs };
99
128
  };
@@ -17,6 +17,19 @@ export interface DroppedOverrideSummary {
17
17
  /** Total override sections dropped across every env + flag. */
18
18
  total: number;
19
19
  }
20
+ /**
21
+ * qfg-gpnd: Per-config tally of rule values that were coerced from a Launch
22
+ * "no value set yet" sentinel ({type:'string', value:''}) to the typed default
23
+ * because the surrounding config valueType was non-string. Tracking is per
24
+ * envId (or 'default' for the default section) so customers can see which
25
+ * environments were affected.
26
+ */
27
+ export interface CoercedSentinelSummary {
28
+ /** Per-envId → per-output-path count of coerced sentinel rule values. */
29
+ byEnv: Record<string, Record<string, number>>;
30
+ /** Total rule values coerced across every env + config. */
31
+ total: number;
32
+ }
20
33
  export interface SkippedConfigEntry {
21
34
  /** Source key (pre-translate) that was skipped. */
22
35
  key: string;
@@ -45,6 +58,12 @@ export interface DuplicateResolutionSummary {
45
58
  }
46
59
  export interface MigrationSource {
47
60
  fetchChanges(sinceEpochMs: null | number): AsyncIterable<LegacyChange>;
61
+ /**
62
+ * Optional post-translate accumulator. Returns any rule values that translate()
63
+ * coerced from a sentinel like Launch's empty-string "no value set yet" to the
64
+ * typed default. Null when nothing was coerced.
65
+ */
66
+ getCoercedSentinels?(): CoercedSentinelSummary | null;
48
67
  /**
49
68
  * Optional post-translate accumulator. Returns any override sections that were
50
69
  * dropped during translate() calls because the env.id was not present in the
@@ -11,7 +11,7 @@ export declare class InvalidSourceConfigError extends Error {
11
11
  export declare function slugify(value: string): string;
12
12
  export declare function isLegacyLogLevel(type: string): boolean;
13
13
  export declare function normalizeLogLevelKey(key: string): string;
14
- export declare function transformConfig(config: LaunchConfig, envIdMap: Record<string, string>, onDroppedEnv?: (envId: string) => void): Record<string, unknown>;
14
+ export declare function transformConfig(config: LaunchConfig, envIdMap: Record<string, string>, onDroppedEnv?: (envId: string) => void, onCoercedSentinel?: (envId: string, valueType: string) => void): Record<string, unknown>;
15
15
  export declare function getOutputPath(type: string, key: string): string;
16
16
  /**
17
17
  * Walks the set of written files looking for cross-type collisions (the same
@@ -102,6 +102,30 @@ function normalizeRules(rules, configKey, path) {
102
102
  }
103
103
  }
104
104
  }
105
+ // qfg-gpnd: Launch emits {type:'string', value:''} as a "no value set yet"
106
+ // sentinel even when the surrounding config valueType is non-string (most
107
+ // commonly seen on catch-all/ALWAYS_TRUE rules). The qfg-verify hook rejects
108
+ // these as type-mismatches and fail-stops the entire push. Coerce the sentinel
109
+ // to the typed default so the otherwise-valid config still ships, and surface
110
+ // each coercion via the callback so the migrator can report it.
111
+ function coerceSentinelRules(rules, valueType, onCoerced) {
112
+ if (!Array.isArray(rules))
113
+ return;
114
+ if (valueType === 'string')
115
+ return;
116
+ for (const rule of rules) {
117
+ if (!rule || typeof rule !== 'object')
118
+ continue;
119
+ const r = rule;
120
+ const v = r.value;
121
+ if (!v || typeof v !== 'object')
122
+ continue;
123
+ if (v.type === 'string' && v.value === '') {
124
+ r.value = zeroValue(valueType);
125
+ onCoerced();
126
+ }
127
+ }
128
+ }
105
129
  function normalizeJsonValuesInConfig(out) {
106
130
  const configKey = String(out.key ?? 'unknown');
107
131
  const defaultSection = out.default;
@@ -122,7 +146,7 @@ function normalizeJsonValuesInConfig(out) {
122
146
  }
123
147
  }
124
148
  }
125
- export function transformConfig(config, envIdMap, onDroppedEnv) {
149
+ export function transformConfig(config, envIdMap, onDroppedEnv, onCoercedSentinel) {
126
150
  const out = JSON.parse(JSON.stringify(config));
127
151
  delete out.changedBy;
128
152
  delete out.modifiedAt;
@@ -137,6 +161,13 @@ export function transformConfig(config, envIdMap, onDroppedEnv) {
137
161
  if (typeof out.valueType === 'string' && out.valueType.toUpperCase() === 'LOG_LEVEL_V2') {
138
162
  out.valueType = 'log_level';
139
163
  }
164
+ // qfg-ol8y: Launch passes through `valueType: 'not_set_value_type'` for
165
+ // segments where the field was never explicitly set. qfg-verify rejects
166
+ // that since segments must have valueType 'bool'. Segments in qfg are
167
+ // unconditionally boolean ("is the user in the segment?"), so force-set.
168
+ if (typeof out.type === 'string' && out.type.toLowerCase() === 'segment') {
169
+ out.valueType = 'bool';
170
+ }
140
171
  if (out.type === 'log_level' && typeof out.key === 'string') {
141
172
  out.key = normalizeLogLevelKey(out.key);
142
173
  }
@@ -199,6 +230,19 @@ export function transformConfig(config, envIdMap, onDroppedEnv) {
199
230
  ],
200
231
  };
201
232
  }
233
+ if (typeof out.valueType === 'string') {
234
+ const valueType = out.valueType;
235
+ const defaultSection = out.default;
236
+ if (defaultSection && Array.isArray(defaultSection.rules)) {
237
+ coerceSentinelRules(defaultSection.rules, valueType, () => onCoercedSentinel?.('default', valueType));
238
+ }
239
+ if (Array.isArray(out.environments)) {
240
+ for (const env of out.environments) {
241
+ const envId = typeof env.id === 'string' ? env.id : 'unknown';
242
+ coerceSentinelRules(env.rules, valueType, () => onCoercedSentinel?.(envId, valueType));
243
+ }
244
+ }
245
+ }
202
246
  normalizeJsonValuesInConfig(out);
203
247
  return out;
204
248
  }
@@ -1,7 +1,13 @@
1
1
  import { fetchAllChangeHistory, fetchEnvironments } from './launch/api.js';
2
2
  import { InvalidSourceConfigError, getOutputPath, isLegacyLogLevel, slugify, transformConfig, } from './launch/translate.js';
3
3
  const SOURCE_NAME = 'launch';
4
- const state = { apiKey: null, droppedOverrides: new Map(), envIdMap: null, skippedConfigs: [] };
4
+ const state = {
5
+ apiKey: null,
6
+ coercedSentinels: new Map(),
7
+ droppedOverrides: new Map(),
8
+ envIdMap: null,
9
+ skippedConfigs: [],
10
+ };
5
11
  class MissingAuthError extends Error {
6
12
  constructor(operation) {
7
13
  super(`launch source ${operation} requires validateAuth(apiKey) to be called first (no API key configured).`);
@@ -58,6 +64,13 @@ function translateImpl(change) {
58
64
  state.droppedOverrides.set(envId, perFlag);
59
65
  }
60
66
  perFlag.set(outputPath, (perFlag.get(outputPath) ?? 0) + 1);
67
+ }, (envId) => {
68
+ let perFlag = state.coercedSentinels.get(envId);
69
+ if (!perFlag) {
70
+ perFlag = new Map();
71
+ state.coercedSentinels.set(envId, perFlag);
72
+ }
73
+ perFlag.set(outputPath, (perFlag.get(outputPath) ?? 0) + 1);
61
74
  });
62
75
  }
63
76
  catch (error) {
@@ -85,6 +98,22 @@ function getDroppedOverridesImpl() {
85
98
  }
86
99
  return { byEnv, total };
87
100
  }
101
+ function getCoercedSentinelsImpl() {
102
+ if (state.coercedSentinels.size === 0)
103
+ return null;
104
+ const byEnv = {};
105
+ let total = 0;
106
+ for (const envId of [...state.coercedSentinels.keys()].sort()) {
107
+ const perFlag = state.coercedSentinels.get(envId);
108
+ const record = {};
109
+ for (const [flagPath, count] of [...perFlag.entries()].sort(([a], [b]) => a.localeCompare(b))) {
110
+ record[flagPath] = count;
111
+ total += count;
112
+ }
113
+ byEnv[envId] = record;
114
+ }
115
+ return { byEnv, total };
116
+ }
88
117
  function getSkippedConfigsImpl() {
89
118
  if (state.skippedConfigs.length === 0)
90
119
  return null;
@@ -95,11 +124,13 @@ async function validateAuthImpl(apiKey) {
95
124
  state.apiKey = apiKey;
96
125
  state.droppedOverrides = new Map();
97
126
  state.skippedConfigs = [];
127
+ state.coercedSentinels = new Map();
98
128
  }
99
129
  export const launchSource = {
100
130
  fetchChanges(sinceEpochMs) {
101
131
  return fetchChangesImpl(sinceEpochMs);
102
132
  },
133
+ getCoercedSentinels: getCoercedSentinelsImpl,
103
134
  getDroppedOverrides: getDroppedOverridesImpl,
104
135
  getSkippedConfigs: getSkippedConfigsImpl,
105
136
  listEnvironments: listEnvironmentsImpl,
@@ -112,4 +143,5 @@ export function __resetLaunchSourceForTests() {
112
143
  state.envIdMap = null;
113
144
  state.droppedOverrides = new Map();
114
145
  state.skippedConfigs = [];
146
+ state.coercedSentinels = new Map();
115
147
  }
@@ -42,6 +42,23 @@ export interface GitOps {
42
42
  getRemoteOriginUrl(dir: string): Promise<string | undefined>;
43
43
  /** Returns true if the dir has a `.git/` (worktree or repo). */
44
44
  isGitRepo(dir: string): Promise<boolean>;
45
+ /**
46
+ * Returns true if origin/main has commits the local HEAD does not have
47
+ * (local strictly behind, or diverged). False when local is up-to-date
48
+ * or strictly ahead of origin/main, or when origin/main is unknown.
49
+ *
50
+ * The clone-path stale-HEAD guard (qfg-fboj) refuses to push in either
51
+ * "behind" or "diverged" state because both produce a diff that ships
52
+ * REVERSAL deltas to the server, silently undoing remote-newer commits.
53
+ */
54
+ isLocalBehindRemote(dir: string): Promise<boolean>;
55
+ /**
56
+ * Tracked files (relative paths) with working-tree or staged changes.
57
+ * Used by the clone-path dirty-tree warning to surface uncommitted
58
+ * edits the user may believe are being pushed (qfg-fboj). Untracked
59
+ * files are excluded — same rule as `dirtyTrackedFiles` in git-ops.ts.
60
+ */
61
+ dirtyTrackedFiles(dir: string): Promise<string[]>;
45
62
  /** Set origin to `url` (add if missing, set-url if present). */
46
63
  setRemoteOrigin(dir: string, url: string): Promise<void>;
47
64
  }
@@ -136,6 +136,15 @@ export async function runPush(input, deps) {
136
136
  await deps.gitOps.setRemoteOrigin(input.dir, backend.repoUrl);
137
137
  log('Fetching from remote...');
138
138
  await deps.gitOps.fetch(input.dir);
139
+ // Guard 4 (qfg-fboj): refuse to push when local HEAD is behind or has
140
+ // diverged from origin/main. Without this guard the HEAD-vs-origin
141
+ // diff produces REVERSAL deltas that silently undo whatever was
142
+ // committed to the cloud since the user last pulled. Tell the user
143
+ // to run `qfg pull` first; the standalone `--ff-only` step keeps
144
+ // push from doing surprising merges on the user's behalf.
145
+ if (await deps.gitOps.isLocalBehindRemote(input.dir)) {
146
+ throw new PushFatalError('Local HEAD is behind origin/main. Run `qfg pull` first to merge remote changes, then re-run `qfg push`.', 'STALE_HEAD');
147
+ }
139
148
  // Migration cleanup (qfg-0fn): older `qfg pull` runs wrote the
140
149
  // workspace pin to the working tree without committing. The next
141
150
  // diff would miss it. Sweep up that legacy state IF the only dirty
@@ -153,6 +162,27 @@ export async function runPush(input, deps) {
153
162
  catch {
154
163
  /* non-fatal */
155
164
  }
165
+ // Guard 5 (qfg-fboj): warn loudly about any tracked file with
166
+ // uncommitted edits beyond the workspace pin. The clone path pushes
167
+ // HEAD content, not the working tree, so these edits are silently
168
+ // dropped from the push. Listing them in the log gives the user a
169
+ // chance to ctrl-C, commit, and retry instead of believing they
170
+ // shipped what's on disk. Untracked files are intentionally not
171
+ // included — same rule as dirtyTrackedFiles in git-ops.ts.
172
+ try {
173
+ const dirty = await deps.gitOps.dirtyTrackedFiles(input.dir);
174
+ const noisyDirty = dirty.filter((f) => f !== 'quonfig.json');
175
+ if (noisyDirty.length > 0) {
176
+ log('');
177
+ log('Warning: working tree has uncommitted changes that will NOT be included in this push:');
178
+ for (const f of noisyDirty)
179
+ log(` ${f}`);
180
+ log('Commit them with `git add` + `git commit` and re-run `qfg push` if you meant to ship them.');
181
+ }
182
+ }
183
+ catch {
184
+ /* non-fatal */
185
+ }
156
186
  }
157
187
  const deltas = await deps.gitOps.diffHeadVsOrigin(input.dir);
158
188
  const totalFilesInRemote = await deps.gitOps.countFilesInRemote(input.dir);
@@ -31,4 +31,7 @@ export interface CloneAndStackPushResult {
31
31
  export declare class PushConflictError extends Error {
32
32
  constructor(message: string);
33
33
  }
34
+ export declare class PushHookRejectedError extends Error {
35
+ constructor(message: string);
36
+ }
34
37
  export declare const cloneAndStackPush: (opts: CloneAndStackPushOptions) => Promise<CloneAndStackPushResult>;
@@ -12,6 +12,12 @@ export class PushConflictError extends Error {
12
12
  this.name = 'PushConflictError';
13
13
  }
14
14
  }
15
+ export class PushHookRejectedError extends Error {
16
+ constructor(message) {
17
+ super(message);
18
+ this.name = 'PushHookRejectedError';
19
+ }
20
+ }
15
21
  const runGit = async (cwd, args) => {
16
22
  try {
17
23
  return await runGitSafe(args, { cwd });
@@ -103,8 +109,15 @@ export const cloneAndStackPush = async (opts) => {
103
109
  const e = error;
104
110
  const stderr = e.stderr ?? '';
105
111
  const combined = `${e.message}\n${stderr}`;
112
+ const detail = redactToken(stderr || e.message);
113
+ // Pre-receive / update hook rejections are functionally distinct from fast-forward
114
+ // conflicts: nothing the user can do by re-fetching will help. Route them to a
115
+ // different lead-in so the hook output (e.g. qfg-verify FAILED ...) is the call to action.
116
+ if (/hook declined|\[remote rejected]/i.test(combined)) {
117
+ throw new PushHookRejectedError(`Push to origin/${branch} was rejected by the remote validation hook. Fix the errors below and push again. Original git error:\n${detail}`);
118
+ }
106
119
  if (/non-fast-forward|fetch first|rejected/i.test(combined)) {
107
- throw new PushConflictError(`Push to origin/${branch} was rejected: the remote has changes we do not have locally. Re-run the migrator to pick up those changes, then push again. Original git error:\n${redactToken(stderr || e.message)}`);
120
+ throw new PushConflictError(`Push to origin/${branch} was rejected: the remote has changes we do not have locally. Re-run the migrator to pick up those changes, then push again. Original git error:\n${detail}`);
108
121
  }
109
122
  throw error;
110
123
  }
@@ -101,6 +101,20 @@ export declare const gitFetch: (dir: string) => Promise<void>;
101
101
  * Returns true if origin/main has commits that can be fast-forwarded into the local branch.
102
102
  */
103
103
  export declare const canFastForward: (dir: string) => Promise<boolean>;
104
+ /**
105
+ * Returns true iff origin/main is NOT an ancestor of the local HEAD —
106
+ * covering both "local strictly behind" and "diverged" in one boolean.
107
+ *
108
+ * Used by the clone-path stale-HEAD guard in `qfg push` (qfg-fboj):
109
+ * either of those two states would otherwise produce a `HEAD..origin/main`
110
+ * diff that ships REVERSAL deltas to the server, silently undoing
111
+ * remote-newer commits. Both must refuse.
112
+ *
113
+ * Returns false on any git failure (no `origin/main` ref yet, no `.git/`,
114
+ * etc.) so the caller falls through to its other guards rather than
115
+ * aborting on an opaque error.
116
+ */
117
+ export declare const isLocalBehindOrDivergedFromRemote: (dir: string) => Promise<boolean>;
104
118
  /**
105
119
  * Returns true if local has commits not reachable from origin/main (diverged).
106
120
  */
@@ -120,3 +134,29 @@ export declare const hasAtLeastOneCommit: (dir: string) => Promise<boolean>;
120
134
  * Get the URL stripped of credentials for display purposes.
121
135
  */
122
136
  export declare const displayUrl: (url: string) => string;
137
+ /**
138
+ * Outcome of `gitPullRebase`. Three branches the caller MUST distinguish
139
+ * — silent failure here is what made qfg-4tey a P1 (qfg pull exit 0 with
140
+ * no recovery path).
141
+ */
142
+ export type GitPullRebaseResult = {
143
+ kind: 'clean';
144
+ commitsRebased: number;
145
+ } | {
146
+ kind: 'conflicts';
147
+ conflictedFiles: string[];
148
+ } | {
149
+ kind: 'failed';
150
+ reason: string;
151
+ };
152
+ /**
153
+ * `git pull --rebase origin main`. Replays local commits on top of the
154
+ * remote tip. On conflicts, leaves the repo in rebase-in-progress state
155
+ * with `<<<<<<<` / `=======` / `>>>>>>>` markers planted by git so the
156
+ * user can resolve via standard git tools (`git rebase --continue` /
157
+ * `git rebase --abort`).
158
+ *
159
+ * Caller is responsible for surfacing recovery instructions; this function
160
+ * only reports the outcome.
161
+ */
162
+ export declare const gitPullRebase: (dir: string) => Promise<GitPullRebaseResult>;
@@ -222,6 +222,41 @@ export const canFastForward = async (dir) => {
222
222
  return false;
223
223
  }
224
224
  };
225
+ /**
226
+ * Returns true iff origin/main is NOT an ancestor of the local HEAD —
227
+ * covering both "local strictly behind" and "diverged" in one boolean.
228
+ *
229
+ * Used by the clone-path stale-HEAD guard in `qfg push` (qfg-fboj):
230
+ * either of those two states would otherwise produce a `HEAD..origin/main`
231
+ * diff that ships REVERSAL deltas to the server, silently undoing
232
+ * remote-newer commits. Both must refuse.
233
+ *
234
+ * Returns false on any git failure (no `origin/main` ref yet, no `.git/`,
235
+ * etc.) so the caller falls through to its other guards rather than
236
+ * aborting on an opaque error.
237
+ */
238
+ export const isLocalBehindOrDivergedFromRemote = async (dir) => {
239
+ try {
240
+ const { stdout: localSha } = await runGit(['-C', dir, 'rev-parse', 'HEAD']);
241
+ const { stdout: remoteSha } = await runGit(['-C', dir, 'rev-parse', 'origin/main']);
242
+ const local = localSha.trim();
243
+ const remote = remoteSha.trim();
244
+ if (local === remote)
245
+ return false;
246
+ try {
247
+ // exit 0 → remote is an ancestor of local → local is strictly ahead, fine.
248
+ await runGit(['-C', dir, 'merge-base', '--is-ancestor', remote, local]);
249
+ return false;
250
+ }
251
+ catch {
252
+ // non-zero exit → remote is NOT an ancestor → behind or diverged.
253
+ return true;
254
+ }
255
+ }
256
+ catch {
257
+ return false;
258
+ }
259
+ };
225
260
  /**
226
261
  * Returns true if local has commits not reachable from origin/main (diverged).
227
262
  */
@@ -310,3 +345,50 @@ export const hasAtLeastOneCommit = async (dir) => {
310
345
  * Get the URL stripped of credentials for display purposes.
311
346
  */
312
347
  export const displayUrl = (url) => redactToken(url);
348
+ /**
349
+ * `git pull --rebase origin main`. Replays local commits on top of the
350
+ * remote tip. On conflicts, leaves the repo in rebase-in-progress state
351
+ * with `<<<<<<<` / `=======` / `>>>>>>>` markers planted by git so the
352
+ * user can resolve via standard git tools (`git rebase --continue` /
353
+ * `git rebase --abort`).
354
+ *
355
+ * Caller is responsible for surfacing recovery instructions; this function
356
+ * only reports the outcome.
357
+ */
358
+ export const gitPullRebase = async (dir) => {
359
+ // Count local-only commits BEFORE the rebase so the success path can
360
+ // tell the user how many got moved. After a clean rebase, origin/main
361
+ // is the merge-base and ahead-count == commits-rebased.
362
+ let commitsRebased = 0;
363
+ try {
364
+ const { stdout } = await runGit(['-C', dir, 'rev-list', '--count', 'origin/main..HEAD']);
365
+ commitsRebased = Number.parseInt(stdout.trim(), 10) || 0;
366
+ }
367
+ catch {
368
+ // Non-fatal — counting is for UX, not correctness.
369
+ }
370
+ try {
371
+ await runGit(['-C', dir, 'pull', '--rebase', 'origin', 'main']);
372
+ return { kind: 'clean', commitsRebased };
373
+ }
374
+ catch (err) {
375
+ // Distinguish "rebase paused on conflicts" from "rebase never started".
376
+ // Conflicts: `diff --diff-filter=U` lists the unmerged paths and
377
+ // `.git/rebase-merge/` is present.
378
+ try {
379
+ const { stdout } = await runGit(['-C', dir, 'diff', '--name-only', '--diff-filter=U']);
380
+ const conflictedFiles = stdout
381
+ .split('\n')
382
+ .map((s) => s.trim())
383
+ .filter(Boolean);
384
+ if (conflictedFiles.length > 0) {
385
+ return { kind: 'conflicts', conflictedFiles };
386
+ }
387
+ }
388
+ catch {
389
+ // Fall through to 'failed' — diff itself errored.
390
+ }
391
+ const reason = err instanceof Error ? err.message : String(err);
392
+ return { kind: 'failed', reason };
393
+ }
394
+ };
package/dist/version.d.ts CHANGED
@@ -1,2 +1,2 @@
1
- declare const _default: "0.0.32";
1
+ declare const _default: "0.0.34";
2
2
  export default _default;
package/dist/version.js CHANGED
@@ -1,2 +1,2 @@
1
1
  // AUTO-GENERATED from package.json by scripts/generate-version.mjs — do not edit.
2
- export default '0.0.32';
2
+ export default '0.0.34';
@@ -1415,6 +1415,12 @@
1415
1415
  "multiple": false,
1416
1416
  "type": "option"
1417
1417
  },
1418
+ "rebase": {
1419
+ "description": "Replay local commits on top of origin/main when the two have diverged. Conflicts are surfaced via standard git markers; resolve with `git rebase --continue`.",
1420
+ "name": "rebase",
1421
+ "allowNo": false,
1422
+ "type": "boolean"
1423
+ },
1418
1424
  "workspace": {
1419
1425
  "description": "Workspace ID (defaults to active profile)",
1420
1426
  "name": "workspace",
@@ -2910,5 +2916,5 @@
2910
2916
  ]
2911
2917
  }
2912
2918
  },
2913
- "version": "0.0.32"
2919
+ "version": "0.0.34"
2914
2920
  }
package/package.json CHANGED
@@ -1,7 +1,7 @@
1
1
  {
2
2
  "packageManager": "yarn@4.11.0+sha512.4e54aeace9141df2f0177c266b05ec50dc044638157dae128c471ba65994ac802122d7ab35bcd9e81641228b7dcf24867d28e750e0bcae8a05277d600008ad54",
3
3
  "name": "@quonfig/cli",
4
- "version": "0.0.32",
4
+ "version": "0.0.34",
5
5
  "bugs": {
6
6
  "url": "https://github.com/quonfig/cli/issues"
7
7
  },