feat:Brevo function

[satyamdev10]

Summary

Add the full brevo function (alias brevo fn) command surface for creating, managing, and deploying Brevo Functions from the CLI.

New commands

• brevo function list — List all Brevo Functions on the account (supports --json, --drafts)
• brevo function get — Show details of a single function by --id or interactive selection
• brevo function activate — Activate a function
• brevo function deactivate — Deactivate a function
• brevo function delete — Delete a function (with confirmation prompt, --force to skip)
• brevo fn init — Interactive AI-powered function creation: describe intent, pick a template or generate from scratch, preview with sample contacts, iterate, and deploy
• brevo fn deploy — Deploy a draft function to production and optionally link it to an app

Key implementation details

• All commands support --json for scripted/CI usage and --id for non-interactive targeting
• Shared helpers in function-action.ts (resolveFunctionId, withNotFoundHandling, executeFunctionAction, buildFunctionActionCommand) eliminate duplication across action commands
• select-function.ts provides reusable interactive function picker with assertFunctionSelectionAllowed guard for non-TTY contexts
• deploy-helpers.ts extracts shared deploy logic (app selection, linking) to reduce cognitive complexity
• preview-table.ts renders function preview results as formatted tables
• SSE streaming support added to api/sse-stream.ts for real-time AI generation feedback during fn init
• 403 handling shows a friendly "Functions not enabled" message when the account lacks the feature
• GA release: preview gate removed — all function commands ship in the published build

Smoke tests

• Full function suite added to the smoke test runner covering: list, get, activate/deactivate cycle, deploy, init (with PTY interaction), and cleanup
• Suite registered in smoke.yml for CI (pre-merge, post-merge, and manual dispatch)

Other changes

• New user-facing strings in src/lang/en.ts for all function commands
• CLI constants added to src/lib/constants.ts
• FEATURE_STAGE['function'] set to 'ga' in src/lib/preview.ts
• functionService added to src/container.ts and src/services/function.ts
• App config template updated to include function-related fields

Test plan

• All 70 test suites pass (1534 tests)
• ESLint clean
• TypeScript build clean
• Smoke test suite covers all function commands
• SonarCloud quality gate (duplication reduced via shared helpers in PRs fix: resolve SonarCloud duplication in function action commands #101, fix: reduce SonarCloud duplication in function commands and smoke tests #102)

[piyushsarin-sib]

Review

Overall the brevo function group is well built: consistent --id / picker / non-TTY / --json handling across every subcommand, a clean standalone SSE reader, and good unit coverage on the function commands. Requesting changes because the third app type is only half-wired into the CLI, and two repo rules (changeset, agent docs) are not met. Details inline; the two findings with no single anchor line are below.

No changeset, no agent-doc update (blocking)

.changeset/ has no pending file for this branch (one was explicitly removed in chore: remove changeset file), and neither agent-context/SKILL.md nor agent-context/AGENTS.md mention brevo function, the fn alias, the new Brevo Function choice in app create, or the "init requires an interactive terminal" rule. CLAUDE.md makes both hard requirements for new user-facing commands, and an installed skill will otherwise keep telling agents these commands do not exist. Please add a minor changeset and document the surface in both files.

Recommendations

• Test the app create Function path end to end. The only create test change asserts the prompt shows three choices. Nothing asserts the brevo_function: {} payload, the rendered app-config.json, or that the OAuth feature scaffold is not offered to a Function app. An app upload test on a scaffolded Function config would have caught the registry gap flagged inline.
• CLAUDE.md's app-type text is now stale. It still describes "the two app types" and the registry comment says "three, soon". Once the registry gains the Function type, add a short Brevo Functions section and update the discriminator wording.
• Fill in the PR body. It is empty. With ~40 commits from eight sub-PRs squashed in, reviewers need the surface summary, the new BREVO_TEST_API_KEY_FUNCTION secret requirement, and the dedicated smoke account dependency in writing.
• SonarCloud gate is red on new duplicated-lines density (7.7% vs 3%). The inline notes on function-action.ts and the duplicated message tables should bring it under.

[piyushsarin-sib]

Went back through the 13 review threads. All are marked resolved and PR #103 is merged into this branch, so the work did land — but three of the replies claim more than what shipped, and I've re-opened those inline. One repo-rule blocker and one CI note below.

Blocker: agent docs not updated

agent-context/SKILL.md and agent-context/AGENTS.md are untouched by this PR — neither mentions brevo function or brevo fn anywhere. CLAUDE.md is explicit that new commands are user-visible behaviour and both files must be updated in the same PR, and that a feature going GA has to land its reference docs in the GA PR. This adds seven commands, a group alias, and a third app type straight to GA, so it needs both files before merge.

SonarCloud: the gate is red, but the analysis is stale

The gate reports new_duplicated_lines_density at 7.7% against a 3% threshold — the same figure as before the dedup work. SonarCloud's last analysis on this PR is dated 2026-09-03 at commit b2134ac, which predates #101, #102 and #103. So the dedup hasn't actually been measured yet and the checked "SonarCloud quality gate" box in the description isn't supported either way. Worth re-running before merge so we know whether it's fixed.

Re-opened inline (was marked resolved)

1. High — brevo app upload is still broken for a Function app. The registry half landed; upload.ts is unchanged and still branches on isUiAppConfig.
2. Medium — only one of the two list reads was paginated; drafts still cap at 50, which makes fn deploy --id report a false not-found.
3. Medium — the SSE 401 mapping. ensureFresh landed, the 401 path didn't.

New

• The new pagination loop can spin forever on a short page; has_more is still unread.
• detectRecord assumes the server echoes brevo_function on the app record — worth confirming, since that's what decides whether app list and scaffold bootstrap treat a Function app correctly.
• SSE reader is released without being cancelled.
• oauthAppType.detectRecord gained a condition no caller exercises.
• --json on function init can only error.

Nits

• The description says --drafts; the flag is --draft.
• Three user-facing strings live outside src/lang/en.ts: Description: (init.ts:324), Name: (deploy-helpers.ts:145), 'Untitled Function' (deploy.ts:75). Spinner labels are inline elsewhere in the repo so those are fine as-is; these three aren't.
• help.ts's gatedSection('brevo-function-type', ...) — the comment now honestly says the definitions carry no requires, so the emergency flip back to preview it describes wouldn't actually gate the commands. Fixing the comment rather than the gate is defensible, but it's worth making that call explicitly rather than by omission.
• fn init's iterate loop offers only update / save, with no way out short of Ctrl-C.

[sonarqubecloud]

Quality Gate failed

Failed conditions
6.5% Duplication on New Code (required ≤ 3%)

See analysis details on SonarQube Cloud

[piyushsarin-sib]

LGTM