digital-tools 2.1.3 → 2.4.0

package/src/providers/messaging/twilio-sms.ts

@@ -56,13 +56,18 @@ export function createTwilioSmsProvider(config: ProviderConfig): SmsProvider {
   ): Promise<any> {
     const url = `${TWILIO_API_URL}/Accounts/${accountSid}${path}`
 
+    const headers: Record<string, string> = {
+      Authorization: getAuthHeader(),
+    }
+    if (body) {
+      headers['Content-Type'] = 'application/x-www-form-urlencoded'
+    }
+
+    const bodyStr = body?.toString()
     const response = await fetch(url, {
       method,
-      headers: {
-        Authorization: getAuthHeader(),
-        ...(body && { 'Content-Type': 'application/x-www-form-urlencoded' }),
-      },
-      body: body?.toString(),
+      headers,
+      ...(bodyStr !== undefined && { body: bodyStr }),
     })
 
     return response.json()
@@ -72,10 +77,10 @@ export function createTwilioSmsProvider(config: ProviderConfig): SmsProvider {
     info: twilioSmsInfo,
 
     async initialize(cfg: ProviderConfig): Promise<void> {
-      accountSid = cfg.accountSid as string
-      authToken = cfg.authToken as string
-      defaultFrom = cfg.defaultFrom as string | undefined
-      messagingServiceSid = cfg.messagingServiceSid as string | undefined
+      accountSid = cfg['accountSid'] as string
+      authToken = cfg['authToken'] as string
+      defaultFrom = cfg['defaultFrom'] as string | undefined
+      messagingServiceSid = cfg['messagingServiceSid'] as string | undefined
 
       if (!accountSid || !authToken) {
         throw new Error('Twilio account SID and auth token are required')
@@ -112,7 +117,10 @@ export function createTwilioSmsProvider(config: ProviderConfig): SmsProvider {
       if (!from && !messagingServiceSid) {
         return {
           success: false,
-          error: { code: 'MISSING_FROM', message: 'From number or messaging service SID is required' },
+          error: {
+            code: 'MISSING_FROM',
+            message: 'From number or messaging service SID is required',
+          },
         }
       }
 
@@ -164,7 +172,10 @@ export function createTwilioSmsProvider(config: ProviderConfig): SmsProvider {
       if (!from && !messagingServiceSid) {
         return {
           success: false,
-          error: { code: 'MISSING_FROM', message: 'From number or messaging service SID is required' },
+          error: {
+            code: 'MISSING_FROM',
+            message: 'From number or messaging service SID is required',
+          },
         }
       }
 
@@ -240,10 +251,16 @@ export function createTwilioSmsProvider(config: ProviderConfig): SmsProvider {
         params.append('From', options.from)
       }
       if (options?.since) {
-        params.append('DateSent>', options.since.toISOString().split('T')[0])
+        const sincePart = options.since.toISOString().split('T')[0]
+        if (sincePart) {
+          params.append('DateSent>', sincePart)
+        }
       }
       if (options?.until) {
-        params.append('DateSent<', options.until.toISOString().split('T')[0])
+        const untilPart = options.until.toISOString().split('T')[0]
+        if (untilPart) {
+          params.append('DateSent<', untilPart)
+        }
       }
 
       const queryString = params.toString()
@@ -281,6 +298,9 @@ function mapTwilioStatus(status: string): SmsStatus['status'] {
 }
 
 function mapTwilioMessage(msg: any): SmsData {
+  const sentAt = msg.date_sent ? new Date(msg.date_sent) : undefined
+  const deliveredAt = msg.status === 'delivered' ? new Date(msg.date_updated) : undefined
+
   return {
     id: msg.sid,
     to: msg.to,
@@ -288,8 +308,8 @@ function mapTwilioMessage(msg: any): SmsData {
     body: msg.body,
     status: msg.status,
     direction: msg.direction === 'inbound' ? 'inbound' : 'outbound',
-    sentAt: msg.date_sent ? new Date(msg.date_sent) : undefined,
-    deliveredAt: msg.status === 'delivered' ? new Date(msg.date_updated) : undefined,
+    ...(sentAt !== undefined && { sentAt }),
+    ...(deliveredAt !== undefined && { deliveredAt }),
   }
 }
 

package/src/providers/project-management/linear.ts

@@ -23,6 +23,93 @@ import { defineProvider } from '../registry.js'
 
 const LINEAR_API_URL = 'https://api.linear.app/graphql'
 
+// =============================================================================
+// Linear API Response Types
+// =============================================================================
+
+/** Linear GraphQL response wrapper */
+interface LinearGraphQLResponse<T> {
+  data?: T
+  errors?: Array<{ message: string }>
+}
+
+/** Linear project from API */
+interface LinearProject {
+  id: string
+  key: string
+  name: string
+  description?: string
+  lead?: { id: string }
+  url: string
+}
+
+/** Linear issue state from API */
+interface LinearIssueState {
+  name: string
+}
+
+/** Linear label from API */
+interface LinearLabel {
+  name: string
+}
+
+/** Linear user reference from API */
+interface LinearUserRef {
+  id: string
+}
+
+/** Linear issue from API */
+interface LinearIssue {
+  id: string
+  identifier: string
+  title: string
+  description?: string
+  state: LinearIssueState
+  priority: number
+  priorityLabel: string
+  labels: { nodes: LinearLabel[] }
+  assignee?: LinearUserRef
+  creator?: LinearUserRef
+  createdAt: string
+  updatedAt: string
+  url: string
+}
+
+/** Linear comment from API */
+interface LinearComment {
+  id: string
+  body: string
+  user: LinearUserRef
+  createdAt: string
+}
+
+/** Linear page info from API */
+interface LinearPageInfo {
+  hasNextPage: boolean
+  endCursor: string
+}
+
+/** Linear issue filter for GraphQL */
+interface LinearIssueFilter {
+  project?: { id: { eq: string } }
+  state?: { name: { in: string[] } }
+  assignee?: { id: { eq: string } }
+  labels?: { name: { in: string[] } }
+}
+
+/** Linear issue create input */
+interface LinearIssueCreateInput {
+  projectId: string
+  title: string
+  description?: string
+  type?: string
+  priority?: number
+  assigneeId?: string
+  parentId?: string
+  estimate?: number
+  labelIds?: string[]
+}
+
 /**
  * Linear provider info
  */
@@ -40,10 +127,10 @@ export const linearInfo: ProviderInfo = {
 /**
  * GraphQL query helper
  */
-async function graphqlRequest<T = any>(
+async function graphqlRequest<T>(
   apiKey: string,
   query: string,
-  variables?: Record<string, any>
+  variables?: Record<string, unknown>
 ): Promise<T> {
   const response = await fetch(LINEAR_API_URL, {
     method: 'POST',
@@ -54,7 +141,7 @@ async function graphqlRequest<T = any>(
     body: JSON.stringify({ query, variables }),
   })
 
-  const result = (await response.json()) as any
+  const result = (await response.json()) as LinearGraphQLResponse<T>
 
   if (result.errors) {
     throw new Error(result.errors[0]?.message || 'GraphQL query failed')
@@ -83,10 +170,7 @@ export function createLinearProvider(config: ProviderConfig): ProjectManagementP
     async healthCheck(): Promise<ProviderHealth> {
       const start = Date.now()
       try {
-        await graphqlRequest(
-          apiKey,
-          `query { viewer { id name } }`
-        )
+        await graphqlRequest(apiKey, `query { viewer { id name } }`)
 
         return {
           healthy: true,
@@ -135,19 +219,19 @@ export function createLinearProvider(config: ProviderConfig): ProjectManagementP
 
       const data = await graphqlRequest<{
         projects: {
-          nodes: any[]
-          pageInfo: { hasNextPage: boolean; endCursor: string }
+          nodes: LinearProject[]
+          pageInfo: LinearPageInfo
         }
       }>(apiKey, query, { first, after })
 
       return {
-        items: data.projects.nodes.map((p: any) => ({
+        items: data.projects.nodes.map((p) => ({
           id: p.id,
           key: p.key,
           name: p.name,
-          description: p.description,
-          lead: p.lead?.id,
           url: p.url,
+          ...(p.description !== undefined && { description: p.description }),
+          ...(p.lead?.id !== undefined && { lead: p.lead.id }),
         })),
         hasMore: data.projects.pageInfo.hasNextPage,
         nextCursor: data.projects.pageInfo.endCursor,
@@ -171,7 +255,9 @@ export function createLinearProvider(config: ProviderConfig): ProjectManagementP
       `
 
       try {
-        const data = await graphqlRequest<{ project: any }>(apiKey, query, { id: projectId })
+        const data = await graphqlRequest<{ project: LinearProject | null }>(apiKey, query, {
+          id: projectId,
+        })
 
         if (!data.project) {
           return null
@@ -181,9 +267,9 @@ export function createLinearProvider(config: ProviderConfig): ProjectManagementP
           id: data.project.id,
           key: data.project.key,
           name: data.project.name,
-          description: data.project.description,
-          lead: data.project.lead?.id,
           url: data.project.url,
+          ...(data.project.description !== undefined && { description: data.project.description }),
+          ...(data.project.lead?.id !== undefined && { lead: data.project.lead.id }),
         }
       } catch {
         return null
@@ -224,15 +310,15 @@ export function createLinearProvider(config: ProviderConfig): ProjectManagementP
         }
       `
 
-      const input: any = {
+      const input: LinearIssueCreateInput = {
         projectId,
         title: issue.title,
-        description: issue.description,
-        ...(issue.type && { type: issue.type }),
-        ...(issue.priority && { priority: parsePriority(issue.priority) }),
-        ...(issue.assigneeId && { assigneeId: issue.assigneeId }),
-        ...(issue.parentId && { parentId: issue.parentId }),
-        ...(issue.estimate && { estimate: issue.estimate }),
+        ...(issue.description !== undefined && { description: issue.description }),
+        ...(issue.type !== undefined && { type: issue.type }),
+        ...(issue.priority !== undefined && { priority: parsePriority(issue.priority) }),
+        ...(issue.assigneeId !== undefined && { assigneeId: issue.assigneeId }),
+        ...(issue.parentId !== undefined && { parentId: issue.parentId }),
+        ...(issue.estimate !== undefined && { estimate: issue.estimate }),
       }
 
       if (issue.labels?.length) {
@@ -240,7 +326,7 @@ export function createLinearProvider(config: ProviderConfig): ProjectManagementP
       }
 
       const data = await graphqlRequest<{
-        issueCreate: { success: boolean; issue: any }
+        issueCreate: { success: boolean; issue: LinearIssue }
       }>(apiKey, query, { input })
 
       if (!data.issueCreate.success) {
@@ -253,16 +339,16 @@ export function createLinearProvider(config: ProviderConfig): ProjectManagementP
         id: created.id,
         key: created.identifier,
         title: created.title,
-        description: created.description,
         type: issue.type || 'Issue',
         status: created.state.name,
         priority: created.priorityLabel,
-        labels: created.labels.nodes.map((l: any) => l.name),
-        assigneeId: created.assignee?.id,
-        reporterId: created.creator?.id,
+        labels: created.labels.nodes.map((l) => l.name),
         createdAt: new Date(created.createdAt),
         updatedAt: new Date(created.updatedAt),
         url: created.url,
+        ...(created.description !== undefined && { description: created.description }),
+        ...(created.assignee?.id !== undefined && { assigneeId: created.assignee.id }),
+        ...(created.creator?.id !== undefined && { reporterId: created.creator.id }),
       }
     },
 
@@ -298,7 +384,9 @@ export function createLinearProvider(config: ProviderConfig): ProjectManagementP
       `
 
       try {
-        const data = await graphqlRequest<{ issue: any }>(apiKey, query, { id: issueId })
+        const data = await graphqlRequest<{ issue: LinearIssue | null }>(apiKey, query, {
+          id: issueId,
+        })
 
         if (!data.issue) {
           return null
@@ -310,16 +398,16 @@ export function createLinearProvider(config: ProviderConfig): ProjectManagementP
           id: issue.id,
           key: issue.identifier,
           title: issue.title,
-          description: issue.description,
           type: 'Issue',
           status: issue.state.name,
           priority: issue.priorityLabel,
-          labels: issue.labels.nodes.map((l: any) => l.name),
-          assigneeId: issue.assignee?.id,
-          reporterId: issue.creator?.id,
+          labels: issue.labels.nodes.map((l) => l.name),
           createdAt: new Date(issue.createdAt),
           updatedAt: new Date(issue.updatedAt),
           url: issue.url,
+          ...(issue.description !== undefined && { description: issue.description }),
+          ...(issue.assignee?.id !== undefined && { assigneeId: issue.assignee.id }),
+          ...(issue.creator?.id !== undefined && { reporterId: issue.creator.id }),
         }
       } catch {
         return null
@@ -360,7 +448,7 @@ export function createLinearProvider(config: ProviderConfig): ProjectManagementP
         }
       `
 
-      const input: any = {}
+      const input: Partial<LinearIssueCreateInput> = {}
 
       if (updates.title) input.title = updates.title
       if (updates.description !== undefined) input.description = updates.description
@@ -369,7 +457,7 @@ export function createLinearProvider(config: ProviderConfig): ProjectManagementP
       if (updates.estimate !== undefined) input.estimate = updates.estimate
 
       const data = await graphqlRequest<{
-        issueUpdate: { success: boolean; issue: any }
+        issueUpdate: { success: boolean; issue: LinearIssue }
       }>(apiKey, query, { id: issueId, input })
 
       if (!data.issueUpdate.success) {
@@ -382,16 +470,16 @@ export function createLinearProvider(config: ProviderConfig): ProjectManagementP
         id: updated.id,
         key: updated.identifier,
         title: updated.title,
-        description: updated.description,
         type: 'Issue',
         status: updated.state.name,
         priority: updated.priorityLabel,
-        labels: updated.labels.nodes.map((l: any) => l.name),
-        assigneeId: updated.assignee?.id,
-        reporterId: updated.creator?.id,
+        labels: updated.labels.nodes.map((l) => l.name),
         createdAt: new Date(updated.createdAt),
         updatedAt: new Date(updated.updatedAt),
         url: updated.url,
+        ...(updated.description !== undefined && { description: updated.description }),
+        ...(updated.assignee?.id !== undefined && { assigneeId: updated.assignee.id }),
+        ...(updated.creator?.id !== undefined && { reporterId: updated.creator.id }),
       }
     },
 
@@ -423,7 +511,7 @@ export function createLinearProvider(config: ProviderConfig): ProjectManagementP
       const after = options?.cursor
 
       // Build filter
-      const filter: any = { project: { id: { eq: projectId } } }
+      const filter: LinearIssueFilter = { project: { id: { eq: projectId } } }
 
       if (options?.status?.length) {
         filter.state = { name: { in: options.status } }
@@ -475,26 +563,26 @@ export function createLinearProvider(config: ProviderConfig): ProjectManagementP
 
       const data = await graphqlRequest<{
         issues: {
-          nodes: any[]
-          pageInfo: { hasNextPage: boolean; endCursor: string }
+          nodes: LinearIssue[]
+          pageInfo: LinearPageInfo
         }
       }>(apiKey, query, { first, after, filter })
 
       return {
-        items: data.issues.nodes.map((issue: any) => ({
+        items: data.issues.nodes.map((issue) => ({
           id: issue.id,
           key: issue.identifier,
           title: issue.title,
-          description: issue.description,
           type: 'Issue',
           status: issue.state.name,
           priority: issue.priorityLabel,
-          labels: issue.labels.nodes.map((l: any) => l.name),
-          assigneeId: issue.assignee?.id,
-          reporterId: issue.creator?.id,
+          labels: issue.labels.nodes.map((l) => l.name),
           createdAt: new Date(issue.createdAt),
           updatedAt: new Date(issue.updatedAt),
           url: issue.url,
+          ...(issue.description !== undefined && { description: issue.description }),
+          ...(issue.assignee?.id !== undefined && { assigneeId: issue.assignee.id }),
+          ...(issue.creator?.id !== undefined && { reporterId: issue.creator.id }),
         })),
         hasMore: data.issues.pageInfo.hasNextPage,
         nextCursor: data.issues.pageInfo.endCursor,
@@ -545,7 +633,7 @@ export function createLinearProvider(config: ProviderConfig): ProjectManagementP
       `
 
       // Build filter
-      const filter: any = {}
+      const filter: LinearIssueFilter = {}
 
       if (options?.status?.length) {
         filter.state = { name: { in: options.status } }
@@ -561,26 +649,26 @@ export function createLinearProvider(config: ProviderConfig): ProjectManagementP
 
       const data = await graphqlRequest<{
         issueSearch: {
-          nodes: any[]
-          pageInfo: { hasNextPage: boolean; endCursor: string }
+          nodes: LinearIssue[]
+          pageInfo: LinearPageInfo
         }
       }>(apiKey, graphqlQuery, { first, after, query, filter })
 
       return {
-        items: data.issueSearch.nodes.map((issue: any) => ({
+        items: data.issueSearch.nodes.map((issue) => ({
           id: issue.id,
           key: issue.identifier,
           title: issue.title,
-          description: issue.description,
           type: 'Issue',
           status: issue.state.name,
           priority: issue.priorityLabel,
-          labels: issue.labels.nodes.map((l: any) => l.name),
-          assigneeId: issue.assignee?.id,
-          reporterId: issue.creator?.id,
+          labels: issue.labels.nodes.map((l) => l.name),
           createdAt: new Date(issue.createdAt),
           updatedAt: new Date(issue.updatedAt),
           url: issue.url,
+          ...(issue.description !== undefined && { description: issue.description }),
+          ...(issue.assignee?.id !== undefined && { assigneeId: issue.assignee.id }),
+          ...(issue.creator?.id !== undefined && { reporterId: issue.creator.id }),
         })),
         hasMore: data.issueSearch.pageInfo.hasNextPage,
         nextCursor: data.issueSearch.pageInfo.endCursor,
@@ -605,7 +693,7 @@ export function createLinearProvider(config: ProviderConfig): ProjectManagementP
       `
 
       const data = await graphqlRequest<{
-        commentCreate: { success: boolean; comment: any }
+        commentCreate: { success: boolean; comment: LinearComment }
       }>(apiKey, query, {
         input: {
           issueId,