Skip to content

feat: add dedicated integration commands and agent guidance - #239

Merged
alick-liming merged 1 commit into
feat/dedicated-integrationsfrom
codex/dedicated-integrations
Oct 27, 2026
Merged

alick-liming merged 1 commit into
feat/dedicated-integrationsfrom
codex/dedicated-integrations

Conversation

@alick-liming

Copy link
Copy Markdown
Contributor

Summary

Adds eight channel integration-* commands for channel-owned dedicated integrations, alongside the existing integration * commands for public integrations. The bundled Agent Skill and bilingual README explain how ownership determines the API, the different permission requirements, and why permission errors must not switch command families.

Generated commands use the typed SDK, preserve partial updates and explicit empty settings, and keep public integration command names compatible. Depends on the SDK change in flashcatcloud/go-flashduty#109; the dependency is pinned to its pushed commit so the CLI builds independently.

Validation

  • make: formatting, repository-pinned lint (0 issues), full race tests, and build passed.
  • go run ./internal/cmd/skilldoc check: all command cards and examples passed.
  • HTTP regressions cover all eight paths, ownership/key output, required channel IDs, omitted/null/empty settings, and access-denied propagation without API fallback.
  • Read-only development smoke checks passed for public list/detail and dedicated list. Dedicated detail currently returns 401; the development gateway still requires JWT for that endpoint and the six dedicated write endpoints. Their end-to-end App Key validation remains dependent on backend authorization rollout.

@alick-liming
alick-liming marked this pull request as ready for review October 27, 2026 05:11
@alick-liming
alick-liming merged commit 3d8d95b into feat/dedicated-integrations Oct 27, 2026
12 checks passed
@alick-liming
alick-liming deleted the codex/dedicated-integrations branch October 27, 2026 05:11
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