Skip to content

fix(ci): restore deploy-lambda.yml (emptied by cherry-pick conflict resolution)#215

Merged
harishquantamix merged 1 commit into
masterfrom
fix/restore-deploy-lambda-yml
Jun 23, 2026
Merged

fix(ci): restore deploy-lambda.yml (emptied by cherry-pick conflict resolution)#215
harishquantamix merged 1 commit into
masterfrom
fix/restore-deploy-lambda-yml

Conversation

@quantamixsol

Copy link
Copy Markdown
Owner

What happened

PR #214 cherry-pick had a merge conflict in deploy-lambda.yml. During resolution, git show private/master -- file hit a Windows shell path-quoting issue and wrote an empty file instead of the correct content. The empty file was committed and merged, which is why the Deploy Lambda workflow immediately failed with "workflow file issue" on the merge push.

Fix

Restores deploy-lambda.yml from private/master — the full file including:

  • FastAPI _IncludedRouter route-check fix (use app.openapi()['paths'] not flat app.routes)
  • Static mount ordering fix (app.mount after all include_router calls)
  • CI diagnostic logging on openapi() failure

Test plan

  • Deploy Lambda workflow runs without "workflow file issue"
  • graqle-check job passes (Verify Studio routes register step exits 0)
  • deploy job completes and cognigraph-api Lambda is updated
  • Smoke test: GET /studio/api/health returns 200

🤖 Generated with Claude Code

…solution

Cherry-pick conflict resolution wrote an empty file (Windows shell path
quoting issue with git show). Restores the full workflow from private/master
including the FastAPI _IncludedRouter openapi() route-check fix and the
static mount ordering fix.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
@github-actions

Copy link
Copy Markdown

🛡️ GraQle PR Guardian

💥 Blast Radius: 1 module affected

Module Files Changed Risk Level Impact Radius
.github 1 🟡 T2 1

Total blast radius: 1


🏛️ Governance Verdict

⚠️ WARN

  • T2: Advisory warnings detected. Review recommended.

🔍 SHACL Violations

No SHACL violations detected.


🔐 Approval Requirements

This PR requires approval from: T2

  • A Senior Engineer or above must approve.

Approval requirement NOT yet satisfied.


Metric Value
Blast Radius 1
Files Analyzed 1
Blocked 0
SHACL Violations 0
Verdict WARN

🔬 Powered by GraQle PR Guardian v0.1.0 · Scan completed 2026-06-23T16:33:14.340095+00:00

@harishquantamix harishquantamix merged commit 0a41899 into master Jun 23, 2026
15 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants