hvac-marketing-skills/tools/clis/kit.js
Corey Haines 8eaff5e29f fix: add input validation and safe JSON parsing across 13 CLIs
Add missing parameter validation to prevent /path/undefined API calls:
- dub: require --url for links create
- google-search-console: require --sitemap-url for sitemaps submit
- kit: validate IDs and emails for subscribers, forms, sequences, tags, broadcasts
- mailchimp: validate IDs for lists get, campaigns get/create/send, members add, reports get
- resend: validate --from/--to/--subject for send, audience/contact IDs for contacts
- rewardful: validate IDs for affiliates get/update, commissions get, links create
- semrush: require --domain/--phrase for all domain and keyword commands
- sendgrid: validate --from/--to/--subject for send, campaign IDs, email for validate

Wrap bare JSON.parse() calls in try/catch for user-provided JSON:
- dub (--links), ga4 (--params), kit (--fields x4), mixpanel (--properties x2),
  onesignal (--filters), paddle (--scheduled-change, --items x2),
  resend (--emails, --variables x2), segment (--properties, --traits, --events),
  sendgrid (--template-data)

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
2026-02-17 22:44:06 -08:00

232 lines
8 KiB
JavaScript
Executable file

#!/usr/bin/env node
const API_SECRET = process.env.KIT_API_SECRET
const API_KEY = process.env.KIT_API_KEY
const BASE_URL = 'https://api.convertkit.com/v3'
if (!API_SECRET && !API_KEY) {
console.error(JSON.stringify({ error: 'KIT_API_SECRET or KIT_API_KEY environment variable required' }))
process.exit(1)
}
async function api(method, path, body, useSecret = true) {
const url = new URL(`${BASE_URL}${path}`)
if (method === 'GET' || method === 'DELETE') {
if (useSecret && API_SECRET) {
url.searchParams.set('api_secret', API_SECRET)
} else if (useSecret && !API_SECRET) {
return { error: 'KIT_API_SECRET required for this endpoint' }
} else if (API_KEY) {
url.searchParams.set('api_key', API_KEY)
}
}
const opts = {
method,
headers: { 'Content-Type': 'application/json' },
}
if (body && (method === 'POST' || method === 'PUT')) {
const authBody = { ...body }
if (useSecret && API_SECRET) {
authBody.api_secret = API_SECRET
} else if (useSecret && !API_SECRET) {
return { error: 'KIT_API_SECRET required for this endpoint' }
} else if (API_KEY) {
authBody.api_key = API_KEY
}
opts.body = JSON.stringify(authBody)
}
if (args['dry-run']) {
const dryRunHeaders = { ...opts.headers }
const dryRunUrl = url.toString().replace(API_SECRET, '***').replace(API_KEY, '***')
let dryRunBody = undefined
if (opts.body) {
const parsed = JSON.parse(opts.body)
if (parsed.api_secret) parsed.api_secret = '***'
if (parsed.api_key) parsed.api_key = '***'
dryRunBody = parsed
}
return { _dry_run: true, method, url: dryRunUrl, headers: dryRunHeaders, body: dryRunBody }
}
const res = await fetch(url.toString(), opts)
const text = await res.text()
try {
return JSON.parse(text)
} catch {
return { status: res.status, body: text }
}
}
function parseArgs(args) {
const result = { _: [] }
for (let i = 0; i < args.length; i++) {
const arg = args[i]
if (arg.startsWith('--')) {
const key = arg.slice(2)
const next = args[i + 1]
if (next && !next.startsWith('--')) {
result[key] = next
i++
} else {
result[key] = true
}
} else {
result._.push(arg)
}
}
return result
}
const args = parseArgs(process.argv.slice(2))
const [cmd, sub, ...rest] = args._
async function main() {
let result
switch (cmd) {
case 'subscribers':
switch (sub) {
case 'list': {
const params = args.page ? `&page=${args.page}` : ''
result = await api('GET', `/subscribers?${params}`)
break
}
case 'get':
if (!rest[0]) { result = { error: 'Subscriber ID required' }; break }
result = await api('GET', `/subscribers/${rest[0]}`)
break
case 'update': {
if (!rest[0]) { result = { error: 'Subscriber ID required' }; break }
const body = {}
if (args['first-name']) body.first_name = args['first-name']
if (args.fields) {
try { body.fields = JSON.parse(args.fields) } catch { result = { error: 'Invalid JSON in --fields' }; break }
}
result = await api('PUT', `/subscribers/${rest[0]}`, body)
break
}
case 'unsubscribe': {
const body = { email: args.email }
result = await api('PUT', '/unsubscribe', body)
break
}
default:
result = { error: 'Unknown subscribers subcommand. Use: list, get, update, unsubscribe' }
}
break
case 'forms':
switch (sub) {
case 'list':
result = await api('GET', '/forms', null, false)
break
case 'subscribe': {
if (!rest[0]) { result = { error: 'Form ID required' }; break }
if (!args.email) { result = { error: '--email required' }; break }
const formId = rest[0]
const body = { email: args.email }
if (args['first-name']) body.first_name = args['first-name']
if (args.fields) {
try { body.fields = JSON.parse(args.fields) } catch { result = { error: 'Invalid JSON in --fields' }; break }
}
result = await api('POST', `/forms/${formId}/subscribe`, body, false)
break
}
default:
result = { error: 'Unknown forms subcommand. Use: list, subscribe' }
}
break
case 'sequences':
switch (sub) {
case 'list':
result = await api('GET', '/sequences', null, false)
break
case 'subscribe': {
if (!rest[0]) { result = { error: 'Sequence ID required' }; break }
if (!args.email) { result = { error: '--email required' }; break }
const sequenceId = rest[0]
const body = { email: args.email }
if (args['first-name']) body.first_name = args['first-name']
if (args.fields) {
try { body.fields = JSON.parse(args.fields) } catch { result = { error: 'Invalid JSON in --fields' }; break }
}
result = await api('POST', `/sequences/${sequenceId}/subscribe`, body, false)
break
}
default:
result = { error: 'Unknown sequences subcommand. Use: list, subscribe' }
}
break
case 'tags':
switch (sub) {
case 'list':
result = await api('GET', '/tags', null, false)
break
case 'subscribe': {
if (!rest[0]) { result = { error: 'Tag ID required' }; break }
if (!args.email) { result = { error: '--email required' }; break }
const tagId = rest[0]
const body = { email: args.email }
if (args['first-name']) body.first_name = args['first-name']
if (args.fields) {
try { body.fields = JSON.parse(args.fields) } catch { result = { error: 'Invalid JSON in --fields' }; break }
}
result = await api('POST', `/tags/${tagId}/subscribe`, body, false)
break
}
case 'remove': {
if (!rest[0]) { result = { error: 'Tag ID required' }; break }
const tagId = rest[0]
const subscriberId = rest[1] || args['subscriber-id']
if (!subscriberId) { result = { error: 'Subscriber ID required' }; break }
result = await api('DELETE', `/subscribers/${subscriberId}/tags/${tagId}`)
break
}
default:
result = { error: 'Unknown tags subcommand. Use: list, subscribe, remove' }
}
break
case 'broadcasts':
switch (sub) {
case 'list': {
const params = args.page ? `&page=${args.page}` : ''
result = await api('GET', `/broadcasts?${params}`)
break
}
case 'create': {
if (!args.subject || !args.content) { result = { error: '--subject and --content required' }; break }
const body = {
subject: args.subject,
content: args.content,
}
if (args['email-layout-template']) body.email_layout_template = args['email-layout-template']
result = await api('POST', '/broadcasts', body)
break
}
default:
result = { error: 'Unknown broadcasts subcommand. Use: list, create' }
}
break
default:
result = {
error: 'Unknown command',
usage: {
subscribers: 'subscribers [list|get|update|unsubscribe] [id] [--email <email>] [--first-name <name>] [--fields <json>] [--page <n>]',
forms: 'forms [list|subscribe] [form_id] [--email <email>] [--first-name <name>] [--fields <json>]',
sequences: 'sequences [list|subscribe] [sequence_id] [--email <email>] [--first-name <name>] [--fields <json>]',
tags: 'tags [list|subscribe|remove] [tag_id] [subscriber_id] [--email <email>] [--subscriber-id <id>]',
broadcasts: 'broadcasts [list|create] [--subject <subject>] [--content <html>] [--email-layout-template <template>] [--page <n>]',
}
}
}
console.log(JSON.stringify(result, null, 2))
}
main().catch(err => {
console.error(JSON.stringify({ error: err.message }))
process.exit(1)
})