Skip to content

Codex/fix raven status legacy stack - #1

Open
Arunkumar-Annamalai wants to merge 2 commits into
giggsoinc:mainfrom
Giggso-Inc:codex/fix-raven-status-legacy-stack
Open

Codex/fix raven status legacy stack#1
Arunkumar-Annamalai wants to merge 2 commits into
giggsoinc:mainfrom
Giggso-Inc:codex/fix-raven-status-legacy-stack

Conversation

@Arunkumar-Annamalai

@Arunkumar-Annamalai Arunkumar-Annamalai commented Jun 4, 2026

Copy link
Copy Markdown
Collaborator

Summary

Fixes raven_status crashing when .raven/manifest.json uses the legacy flat manifest shape.

Before this change, Raven assumed manifest["stack"] was always an object and called .get("language") / .get("cloud") on it. In projects where the manifest used the older format like:

{
  "stack": "react",
  "cloud": "none"
}

raven_status failed with:

'str' object has no attribute 'get'

This update makes Raven compatible with both manifest formats:

  • legacy flat format: "stack": "react", "cloud": "none"
  • nested format: "stack": { "language": "react", "cloud": "none" }

What changed

  • Added a normalize_stack() helper to normalize manifest data before reading stack fields
  • Updated raven_status to use normalized language and cloud values
  • Applied the same fix to both Raven server entrypoints so Codex and script-based flows behave consistently

Files changed

  • mcp/server.py
  • scripts/server.py

Verification

  • Ran mcp/server.py directly against a legacy flat manifest
  • Ran scripts/server.py directly against a legacy flat manifest
  • Confirmed raven_status now returns success instead of crashing

Result

raven_status now works for both old and new Raven manifest schemas without requiring existing projects to migrate their manifest immediately.

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.

1 participant