Skip to content

Fix Arcane API response parsing to unwrap .data field#14

Merged
nsheaps merged 1 commit intomainfrom
claude/fix-arcane-ci-deployment-UTgU0
Mar 20, 2026
Merged

Fix Arcane API response parsing to unwrap .data field#14
nsheaps merged 1 commit intomainfrom
claude/fix-arcane-ci-deployment-UTgU0

Conversation

@nsheaps
Copy link
Owner

@nsheaps nsheaps commented Mar 20, 2026

Summary

  • Fix action.sh to properly parse Arcane API responses which wrap data in {success, data, pagination} format
  • List endpoints: use .data[] instead of .[] to iterate items from paginated responses
  • Create endpoints: use .data.id instead of .id to extract IDs from wrapped responses

Without this fix, the repository lookup always misses existing repos (causing duplicate creation attempts) and ID extraction from create responses fails, causing the deploy step to error out.

Test plan

  • Re-run the arcane-deploy workflow in nsheaps/iac (manual dispatch with force=true)
  • Verify the deploy step completes successfully
  • Verify existing repos are found and updated (not re-created)

https://claude.ai/code/session_01EPuZkcSGc4zpbDDBvMd3Jz

The Arcane API wraps all responses in {success, data, pagination} but
the action was parsing responses as raw arrays/objects. This caused
repository lookup to always miss (creating duplicates) and ID extraction
from create responses to fail.

- List endpoints: use .data[] instead of .[] to iterate items
- Create endpoints: use .data.id instead of .id to extract IDs

https://claude.ai/code/session_01EPuZkcSGc4zpbDDBvMd3Jz
@nsheaps nsheaps merged commit a257437 into main Mar 20, 2026
2 checks passed
@nsheaps nsheaps deleted the claude/fix-arcane-ci-deployment-UTgU0 branch March 20, 2026 02:45
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