@riddledc/riddle-proof 0.5.28 → 0.5.29

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@riddledc/riddle-proof",
-  "version": "0.5.28",
+  "version": "0.5.29",
   "description": "Reusable Riddle Proof contracts and helpers for evidence-backed agent changes.",
   "license": "MIT",
   "author": "RiddleDC",

package/runtime/lib/preflight.py

@@ -42,6 +42,7 @@ with open(args_file) as f:
 
 mode = (s.get('mode') or '').strip().lower()
 reference = s.get('reference', 'both')
+requested_reference = reference
 reference_note = ''
 verification_mode = (s.get('verification_mode') or 'proof').strip() or 'proof'
 s['verification_mode'] = verification_mode
@@ -68,6 +69,15 @@ if discord_url_match:
 if reference not in ('prod', 'before', 'both'):
     raise SystemExit('Invalid reference: ' + reference + '. Must be prod, before, or both.')
 s['reference'] = reference
+prod_url_present = bool((s.get('prod_url') or '').strip())
+s['reference_resolution'] = {
+    'requested_reference': requested_reference,
+    'effective_reference': reference,
+    'prod_reference_requested': requested_reference in ('prod', 'both'),
+    'prod_url_present': prod_url_present,
+    'prod_reference_skipped': False,
+    'prod_reference_skip_reason': '',
+}
 
 # Infer a reasonable commit title during setup instead of forcing the caller to
 # fill boilerplate that can be derived from the requested change.
@@ -77,11 +87,17 @@ if not (s.get('commit_message') or '').strip():
 # Setup should not block on a missing production URL. If prod comparison was
 # requested but prod_url is not known yet, continue with a before-only setup so
 # the repo homework can happen first.
-if reference in ('prod', 'both') and not (s.get('prod_url') or '').strip():
+if reference in ('prod', 'both') and not prod_url_present:
     s['requested_reference'] = reference
     reference = 'before'
     s['reference'] = reference
     reference_note = 'prod_url not provided; setup will continue with reference=before until prod is known.'
+    s['reference_resolution'].update({
+        'effective_reference': reference,
+        'prod_reference_skipped': True,
+        'prod_reference_skip_reason': 'prod_url_not_provided',
+        'note': reference_note,
+    })
 
 # Parse optional assertions JSON
 parsed_assertions = None

package/runtime/tests/recon_verify_smoke.py

@@ -11,6 +11,7 @@ from pathlib import Path
 ROOT = Path(__file__).resolve().parents[1]
 LIB = ROOT / 'lib'
 UTIL_PATH = LIB / 'util.py'
+PREFLIGHT_PATH = LIB / 'preflight.py'
 RECON_PATH = LIB / 'recon.py'
 VERIFY_PATH = LIB / 'verify.py'
 AUTHOR_PATH = LIB / 'author.py'
@@ -593,6 +594,49 @@ def temporary_env(**updates):
                 os.environ[key] = value
 
 
+def run_preflight_records_prod_reference_skip_reason():
+    tempdir = Path(tempfile.mkdtemp(prefix='riddle-proof-preflight-reference-'))
+    args_path = tempdir / 'args.json'
+    state_path = tempdir / 'state.json'
+    repo_dir = tempdir / 'repo'
+    try:
+        make_project(repo_dir, "export const routes = [{ path: '/pricing', element: <Pricing /> }];\n")
+        args_path.write_text(json.dumps({
+            'repo': 'example/repo',
+            'repo_dir': str(repo_dir),
+            'mode': 'static',
+            'reference': 'both',
+            'prod_url': '',
+            'change_request': 'Make the pricing CTA clearer',
+            'commit_message': 'Make the pricing CTA clearer',
+            'success_criteria': 'Pricing CTA is visible.',
+            'verification_mode': 'text',
+            'build_command': BUILD_SCRIPT,
+            'build_output': 'build',
+            'allow_static_preview_fallback': True,
+            'server_path': '/pricing',
+        }, indent=2))
+        with temporary_env(
+            RIDDLE_PROOF_ARGS_FILE=str(args_path),
+            RIDDLE_PROOF_STATE_FILE=str(state_path),
+        ):
+            load_module('util_preflight_reference_skip', UTIL_PATH)
+            load_module('preflight_reference_skip', PREFLIGHT_PATH)
+        after_preflight = json.loads(state_path.read_text())
+        assert after_preflight['requested_reference'] == 'both'
+        assert after_preflight['reference'] == 'before'
+        assert after_preflight['reference_resolution']['requested_reference'] == 'both'
+        assert after_preflight['reference_resolution']['effective_reference'] == 'before'
+        assert after_preflight['reference_resolution']['prod_reference_skipped'] is True
+        assert after_preflight['reference_resolution']['prod_reference_skip_reason'] == 'prod_url_not_provided'
+        return {
+            'ok': True,
+            'reference_resolution': after_preflight['reference_resolution'],
+        }
+    finally:
+        shutil.rmtree(tempdir, ignore_errors=True)
+
+
 def base_state(tempdir: Path, *, reference='before', prod_url=''):
     before_dir = tempdir / 'before'
     after_dir = tempdir / 'after'
@@ -1740,6 +1784,7 @@ def run_ship_resolves_real_pr_branch():
 
 if __name__ == '__main__':
     payload = {
+        'preflight_reference_skip_reason': run_preflight_records_prod_reference_skip_reason(),
         'capture_artifact_enrichment': run_capture_artifact_enrichment(),
         'capture_diagnostics_redaction': run_capture_diagnostics_redact_sensitive_values(),
         'apply_auth_context': run_apply_auth_context_passes_supported_auth_payloads(),