Skip to content

feat(apps): align +release commands with new release API format - #48

Merged
raistlin042 merged 5 commits into
feat/lark-apps-devfrom
feat/apps-release-api-upgrade
Jun 9, 2026
Merged

feat(apps): align +release commands with new release API format#48
raistlin042 merged 5 commits into
feat/lark-apps-devfrom
feat/apps-release-api-upgrade

Conversation

@raistlin042

Copy link
Copy Markdown
Owner

Summary

The spark app.release OpenAPI was upgraded. This PR aligns the apps +release-* command family with the new format. Response fields remain snake_case at the gateway (the camelCase go.tag values in the new IDL are internal thrift tags, not the public wire format — verified against the existing apps +list command and existing release unit tests), so no field-casing migration is involved. Three surgical deltas: the create scope, the list page-size documented ceiling, and surfacing the new commit_id field.

Changes

  • shortcuts/apps/apps_release_create.go: align +release-create scope spark:app:publish -> spark:app:write
  • shortcuts/apps/apps_release_list.go: raise --page-size documented max from 50 to 500 (server enforces via vt.le:500; default stays 20, no client-side validation added)
  • shortcuts/apps/apps_release_get.go: print a commit_id line in pretty output when non-empty (JSON output already passes it through); placed before the status-conditional online_url/error_logs block, independent of status
  • shortcuts/apps/apps_release_create_test.go, apps_release_get_test.go: update scope assertion; add present/absent/empty commit_id rendering tests
  • skills/lark-apps/references/lark-apps-release-list.md, lark-apps-release-get.md: document the page-size 500 ceiling and the commit_id output field

Test Plan

  • make unit-test passed (go test ./shortcuts/apps/... all green)
  • validate passed (build / vet / unit / integration)
  • local-eval passed (E2E 5/5, skillave N/A — changes introduce no new AI decision point, 0 eval cases)
  • acceptance-reviewer passed (7/7 cases)
  • manual verification: lark-cli apps +release-list --help shows page size (max 500); +release-create --app-id app_x --branch foo --dry-run emits POST /open-apis/spark/v1/apps/app_x/releases with body {"branch":"foo"}; +release-get --app-id app_x --release-id 1 --dry-run emits the correct GET path

Related Issues

N/A

@raistlin042 raistlin042 added the feature New functionality label Jun 9, 2026
@github-actions github-actions Bot added the size/M Single-domain feat or fix with limited business impact label Jun 9, 2026
@raistlin042
raistlin042 merged commit 71eddc3 into feat/lark-apps-dev Jun 9, 2026
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

feature New functionality size/M Single-domain feat or fix with limited business impact

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant