@knpkv/jira-cli 1.1.1 → 1.2.0

package/src/IssueService.ts

@@ -18,6 +18,7 @@
  *
  * @module
  */
+import { normalizeAttachmentMediaType } from "@knpkv/atlassian-common/attachments"
 import { JiraApiClient, toEffect } from "@knpkv/jira-api-client"
 import * as Context from "effect/Context"
 import * as Effect from "effect/Effect"
@@ -40,6 +41,8 @@ export interface Attachment {
   readonly id: string
   readonly filename: string
   readonly url: string
+  readonly mediaType: string | null
+  /** Backwards-compatible alias for consumers that still read Jira's MIME field name. */
   readonly mimeType: string
   readonly size: number
 }
@@ -110,6 +113,24 @@ interface SearchJqlResponse {
   readonly nextPageToken?: string
 }
 
+const isRecord = (value: unknown): value is Readonly<Record<PropertyKey, unknown>> =>
+  typeof value === "object" && value !== null && !Array.isArray(value)
+
+const recordOrEmpty = (value: unknown): Readonly<Record<PropertyKey, unknown>> => isRecord(value) ? value : {}
+
+const recordArray = (value: unknown): ReadonlyArray<Readonly<Record<PropertyKey, unknown>>> =>
+  Array.isArray(value) ? value.filter(isRecord) : []
+
+const parseSearchJqlResponse = (value: unknown): SearchJqlResponse => {
+  const record = recordOrEmpty(value)
+  const nextPageToken = typeof record.nextPageToken === "string" ? record.nextPageToken : undefined
+  return {
+    issues: Array.isArray(record.issues) ? record.issues : [],
+    isLast: typeof record.isLast === "boolean" ? record.isLast : nextPageToken === undefined,
+    ...(nextPageToken !== undefined ? { nextPageToken } : {})
+  }
+}
+
 /**
  * IssueService interface.
  *
@@ -172,10 +193,9 @@ const FIELDS = [
 const extractDisplayName = (field: unknown): string | null => {
   if (field === null || field === undefined) return null
   if (typeof field === "string") return field
-  if (typeof field === "object") {
-    const obj = field as Record<string, unknown>
-    if (typeof obj["displayName"] === "string") return obj["displayName"]
-    if (typeof obj["name"] === "string") return obj["name"]
+  if (isRecord(field)) {
+    if (typeof field.displayName === "string") return field.displayName
+    if (typeof field.name === "string") return field.name
   }
   return null
 }
@@ -188,9 +208,8 @@ const extractNameArray = (field: unknown): ReadonlyArray<string> => {
   return field
     .map((item) => {
       if (typeof item === "string") return item
-      if (typeof item === "object" && item !== null) {
-        const obj = item as Record<string, unknown>
-        if (typeof obj["name"] === "string") return obj["name"]
+      if (isRecord(item)) {
+        if (typeof item.name === "string") return item.name
       }
       return null
     })
@@ -210,34 +229,35 @@ const parseDate = (val: unknown): Date => {
 /**
  * Map IssueBean from API to our Issue type.
  */
-const mapIssue = (bean: Record<string, unknown>, baseUrl: string): Issue => {
-  const fields = (bean["fields"] ?? {}) as Record<string, unknown>
-  const renderedFields = (bean["renderedFields"] ?? {}) as Record<string, unknown>
+const mapIssue = (bean: Readonly<Record<PropertyKey, unknown>>, baseUrl: string): Issue => {
+  const fields = recordOrEmpty(bean.fields)
+  const renderedFields = recordOrEmpty(bean.renderedFields)
   const key = String(bean["key"] ?? "")
   const id = String(bean["id"] ?? "")
 
   // Extract attachments
   const attachmentField = fields["attachment"]
   const attachments: Array<Attachment> = Array.isArray(attachmentField)
-    ? attachmentField.map((a) => {
-      const att = a as Record<string, unknown>
+    ? recordArray(attachmentField).map((att) => {
+      const mediaType = normalizeAttachmentMediaType(
+        undefined,
+        typeof att["mimeType"] === "string" ? att["mimeType"] : null
+      )
       return {
         id: String(att["id"] ?? ""),
         filename: String(att["filename"] ?? ""),
         url: String(att["content"] ?? ""),
-        mimeType: String(att["mimeType"] ?? ""),
+        mediaType,
+        mimeType: mediaType ?? "",
         size: Number(att["size"] ?? 0)
       }
     })
     : []
 
   // Extract comments
-  const commentField = fields["comment"] as Record<string, unknown> | undefined
-  const commentList = (commentField?.["comments"] ?? []) as Array<Record<string, unknown>>
-  const renderedComments =
-    ((renderedFields["comment"] as Record<string, unknown> | undefined)?.["comments"] ?? []) as Array<
-      Record<string, unknown>
-    >
+  const commentField = recordOrEmpty(fields["comment"])
+  const commentList = recordArray(commentField.comments)
+  const renderedComments = recordArray(recordOrEmpty(renderedFields.comment).comments)
 
   // Build map of rendered comments by ID for accurate matching
   const renderedMap = new Map<string, Record<string, unknown>>()
@@ -247,7 +267,7 @@ const mapIssue = (bean: Record<string, unknown>, baseUrl: string): Issue => {
   }
 
   const comments: Array<Comment> = commentList.map((c) => {
-    const author = c["author"] as Record<string, unknown> | undefined
+    const author = recordOrEmpty(c.author)
     const commentId = String(c["id"] ?? "")
     const rendered = renderedMap.get(commentId)
     const renderedBody = rendered?.["body"]
@@ -286,6 +306,8 @@ const mapIssue = (bean: Record<string, unknown>, baseUrl: string): Issue => {
   }
 }
 
+const mapIssueUnknown = (bean: unknown, baseUrl: string): Issue => mapIssue(recordOrEmpty(bean), baseUrl)
+
 const make = Effect.gen(function*() {
   const client = yield* JiraApiClient
   const siteUrl = yield* SiteUrl
@@ -297,7 +319,7 @@ const make = Effect.gen(function*() {
         query: { fields: FIELDS, expand: "renderedFields" }
       }
     })).pipe(
-      Effect.map((result) => mapIssue(result as unknown as Record<string, unknown>, siteUrl)),
+      Effect.map((result) => mapIssueUnknown(result, siteUrl)),
       Effect.mapError((cause) => new JiraApiError({ message: `Failed to get issue ${key}`, cause }))
     )
 
@@ -317,7 +339,7 @@ const make = Effect.gen(function*() {
         }
       }
     })).pipe(
-      Effect.map((result) => result as SearchJqlResponse),
+      Effect.map(parseSearchJqlResponse),
       Effect.mapError((cause) => new JiraApiError({ message: "Failed to search issues", cause }))
     )
 
@@ -327,7 +349,7 @@ const make = Effect.gen(function*() {
         const issues = result.issues ?? []
         const mappedIssues: Array<Issue> = []
         for (const bean of issues) {
-          mappedIssues.push(mapIssue(bean as unknown as Record<string, unknown>, siteUrl))
+          mappedIssues.push(mapIssueUnknown(bean, siteUrl))
         }
         return {
           issues: mappedIssues,
@@ -343,35 +365,36 @@ const make = Effect.gen(function*() {
   const searchAll = (
     jql: string,
     options?: { readonly maxResults?: number }
-  ): Effect.Effect<ReadonlyArray<Issue>, JiraApiError> => (Effect.gen(function*() {
-    const allIssues: Array<Issue> = []
-    const maxResults = options?.maxResults ?? 100
-    let nextPageToken: string | undefined = undefined
-    let pageCount = 0
-
-    // Fetch first page
-    let result = yield* searchJql(jql, maxResults, nextPageToken)
-    let issues = result.issues ?? []
-    pageCount++
-
-    for (const bean of issues) {
-      allIssues.push(mapIssue(bean as unknown as Record<string, unknown>, siteUrl))
-    }
-
-    // Fetch remaining pages with iteration guard
-    while (!result.isLast && result.nextPageToken && pageCount < MAX_PAGES) {
-      nextPageToken = result.nextPageToken
-      result = yield* searchJql(jql, maxResults, nextPageToken)
-      issues = result.issues ?? []
+  ): Effect.Effect<ReadonlyArray<Issue>, JiraApiError> =>
+    Effect.gen(function*() {
+      const allIssues: Array<Issue> = []
+      const maxResults = options?.maxResults ?? 100
+      let nextPageToken: string | undefined = undefined
+      let pageCount = 0
+
+      // Fetch first page
+      let result = yield* searchJql(jql, maxResults, nextPageToken)
+      let issues = result.issues ?? []
       pageCount++
 
       for (const bean of issues) {
-        allIssues.push(mapIssue(bean as unknown as Record<string, unknown>, siteUrl))
+        allIssues.push(mapIssueUnknown(bean, siteUrl))
+      }
+
+      // Fetch remaining pages with iteration guard
+      while (!result.isLast && result.nextPageToken && pageCount < MAX_PAGES) {
+        nextPageToken = result.nextPageToken
+        result = yield* searchJql(jql, maxResults, nextPageToken)
+        issues = result.issues ?? []
+        pageCount++
+
+        for (const bean of issues) {
+          allIssues.push(mapIssueUnknown(bean, siteUrl))
+        }
       }
-    }
 
-    return allIssues
-  }) as Effect.Effect<ReadonlyArray<Issue>, JiraApiError>)
+      return allIssues
+    })
 
   return IssueService.of({ getByKey, search, searchAll })
 })

package/src/JiraAuth.ts

@@ -226,17 +226,16 @@ export class JiraAuth extends Context.Service<
   JiraAuthService
 >()("@knpkv/jira-cli/JiraAuth") {}
 
+type RefreshError = OAuthError | FileSystemError | HomeDirectoryError | PlatformError.PlatformError
+type RefreshDeferred = Deferred.Deferred<OAuthToken, RefreshError>
+
 const make = Effect.gen(function*() {
   const httpClient = yield* HttpClient.HttpClient
   const childProcessSpawner = yield* ChildProcessSpawner.ChildProcessSpawner
   const cryptoService = yield* Crypto.Crypto
 
   // Ref to track ongoing refresh operation to prevent concurrent refreshes
-  const refreshLock = yield* Ref.make<
-    Option.Option<
-      Deferred.Deferred<OAuthToken, OAuthError | FileSystemError | HomeDirectoryError | PlatformError.PlatformError>
-    >
-  >(
+  const refreshLock = yield* Ref.make<Option.Option<RefreshDeferred>>(
     Option.none()
   )
 
@@ -422,14 +421,11 @@ const make = Effect.gen(function*() {
       }
 
       // Atomically check-then-set refresh lock to avoid TOCTOU race
-      const deferred = yield* Deferred.make<
-        OAuthToken,
-        OAuthError | FileSystemError | HomeDirectoryError | PlatformError.PlatformError
-      >()
+      const deferred = yield* Deferred.make<OAuthToken, RefreshError>()
       const existing = yield* Ref.modify(refreshLock, (current) =>
         Option.isSome(current)
-          ? [current.value, current] as const
-          : [deferred, Option.some(deferred)] as const)
+          ? ([current.value, current] satisfies readonly [RefreshDeferred, Option.Option<RefreshDeferred>])
+          : ([deferred, Option.some(deferred)] satisfies readonly [RefreshDeferred, Option.Option<RefreshDeferred>]))
 
       // Another fiber is already refreshing — just await its result
       if (existing !== deferred) {

package/src/VersionService.ts

@@ -198,7 +198,9 @@ const EXPAND = "approvers,driver,operations,issuesstatus,contributors"
 
 /** Loosely-typed record helper for navigating untyped API JSON. */
 type Raw = Record<string, unknown>
-const asRaw = (v: unknown): Raw => (v && typeof v === "object" ? v as Raw : {})
+const isRaw = (value: unknown): value is Raw => typeof value === "object" && value !== null && !Array.isArray(value)
+const asRaw = (value: unknown): Raw => isRaw(value) ? value : {}
+const rawArray = (value: unknown): ReadonlyArray<Raw> => Array.isArray(value) ? value.filter(isRaw) : []
 
 /**
  * Render a Jira custom-field value as a flat string.
@@ -219,13 +221,13 @@ export const renderCustomFieldValue = (raw: unknown): string | null => {
     const parts = raw.map(renderCustomFieldValue).filter((v): v is string => !!v)
     return parts.length > 0 ? parts.join(", ") : null
   }
-  if (typeof raw === "object") {
-    const obj = raw as Raw
+  if (isRaw(raw)) {
+    const obj = raw
     const parent = stringOrNull(obj["value"])
     if (parent) {
       const child = obj["child"]
-      if (child && typeof child === "object") {
-        const childValue = stringOrNull((child as Raw)["value"])
+      if (isRaw(child)) {
+        const childValue = stringOrNull(child["value"])
         if (childValue) return `${parent} > ${childValue}`
       }
       return parent
@@ -241,8 +243,8 @@ export const renderCustomFieldValue = (raw: unknown): string | null => {
 const stringOrNull = (v: unknown): string | null => typeof v === "string" && v.length > 0 ? v : null
 
 export const personFromObject = (raw: unknown, fallbackId?: string): Person | null => {
-  if (raw && typeof raw === "object") {
-    const obj = raw as Raw
+  if (isRaw(raw)) {
+    const obj = raw
     const accountId = stringOrNull(obj["accountId"]) ?? fallbackId ?? null
     if (!accountId) return null
     return {
@@ -266,8 +268,8 @@ export const extractContributorIds = (raw: Raw): ReadonlyArray<string> => {
   const ids: Array<string> = []
   for (const c of field) {
     if (typeof c === "string" && c.length > 0) ids.push(c)
-    else if (c && typeof c === "object") {
-      const id = (c as Raw)["accountId"]
+    else if (isRaw(c)) {
+      const id = c["accountId"]
       if (typeof id === "string" && id.length > 0) ids.push(id)
     }
   }
@@ -297,26 +299,23 @@ const make = Effect.gen(function*() {
 
   const resolveFieldIds = (
     name: string
-  ): Effect.Effect<ReadonlyArray<string>, JiraApiError> => (Effect.gen(function*() {
-    const cached = fieldIdsByName.get(name)
-    if (cached !== undefined) return cached
-    const result = yield* toEffect(
-      client.v3.client.GET("/rest/api/3/field")
-    ).pipe(Effect.mapError((cause) => new JiraApiError({ message: `Failed to list Jira fields`, cause })))
-    const matches: Array<string> = []
-    if (Array.isArray(result)) {
-      for (const f of result) {
-        if (f && typeof f === "object") {
-          const obj = f as Raw
-          if (obj["name"] === name && typeof obj["id"] === "string") {
-            matches.push(obj["id"] as string)
-          }
+  ): Effect.Effect<ReadonlyArray<string>, JiraApiError> =>
+    Effect.gen(function*() {
+      const cached = fieldIdsByName.get(name)
+      if (cached !== undefined) return cached
+      const result = yield* toEffect(
+        client.v3.client.GET("/rest/api/3/field")
+      ).pipe(Effect.mapError((cause) => new JiraApiError({ message: `Failed to list Jira fields`, cause })))
+      const matches: Array<string> = []
+      for (const field of rawArray(result)) {
+        const id = field["id"]
+        if (field["name"] === name && typeof id === "string") {
+          matches.push(id)
         }
       }
-    }
-    fieldIdsByName.set(name, matches)
-    return matches
-  }) as Effect.Effect<ReadonlyArray<string>, JiraApiError>)
+      fieldIdsByName.set(name, matches)
+      return matches
+    })
 
   const fetchUser = (accountId: string): Effect.Effect<Person, never> =>
     toEffect(
@@ -379,95 +378,96 @@ const make = Effect.gen(function*() {
     versionName: string,
     customFieldNames: ReadonlyArray<string>,
     projectKey?: string
-  ): Effect.Effect<ReadonlyArray<VersionTicket>, JiraApiError> => (Effect.gen(function*() {
-    const nameToIds = new Map<string, ReadonlyArray<string>>()
-    for (const name of customFieldNames) {
-      const ids = yield* resolveFieldIds(name)
-      nameToIds.set(name, ids)
-    }
-    const allFieldIds = new Set<string>()
-    for (const ids of nameToIds.values()) for (const id of ids) allFieldIds.add(id)
-    const requestedFields = ["assignee", "labels", "summary", ...allFieldIds]
-
-    const raws: Array<RawTicket> = []
-    const PAGE = 100
-    const MAX_PAGES = 100
-    let nextPageToken: string | undefined = undefined
-    for (let page = 0; page < MAX_PAGES; page++) {
-      const result = yield* toEffect(
-        client.v3.client.GET("/rest/api/3/search/jql", {
-          params: {
-            query: {
-              jql: buildByVersionJql(versionName, projectKey),
-              fields: requestedFields,
-              maxResults: PAGE,
-              ...(nextPageToken ? { nextPageToken } : {})
+  ): Effect.Effect<ReadonlyArray<VersionTicket>, JiraApiError> =>
+    Effect.gen(function*() {
+      const nameToIds = new Map<string, ReadonlyArray<string>>()
+      for (const name of customFieldNames) {
+        const ids = yield* resolveFieldIds(name)
+        nameToIds.set(name, ids)
+      }
+      const allFieldIds = new Set<string>()
+      for (const ids of nameToIds.values()) for (const id of ids) allFieldIds.add(id)
+      const requestedFields = ["assignee", "labels", "summary", ...allFieldIds]
+
+      const raws: Array<RawTicket> = []
+      const PAGE = 100
+      const MAX_PAGES = 100
+      let nextPageToken: string | undefined = undefined
+      for (let page = 0; page < MAX_PAGES; page++) {
+        const result = yield* toEffect(
+          client.v3.client.GET("/rest/api/3/search/jql", {
+            params: {
+              query: {
+                jql: buildByVersionJql(versionName, projectKey),
+                fields: requestedFields,
+                maxResults: PAGE,
+                ...(nextPageToken ? { nextPageToken } : {})
+              }
             }
-          }
-        })
-      ).pipe(
-        Effect.mapError((cause) =>
-          new JiraApiError({ message: `Failed to fetch tickets for fixVersion "${versionName}"`, cause })
+          })
+        ).pipe(
+          Effect.mapError((cause) =>
+            new JiraApiError({ message: `Failed to fetch tickets for fixVersion "${versionName}"`, cause })
+          )
         )
-      )
 
-      const resObj = asRaw(result)
-      const issues = (Array.isArray(resObj["issues"]) ? resObj["issues"] : []) as Array<Raw>
-      for (const issue of issues) {
-        const key = typeof issue["key"] === "string" ? issue["key"] as string : ""
-        const fields = asRaw(issue["fields"])
-        const assignee = fields["assignee"]
-        let assigneeId: string | null = null
-        if (assignee && typeof assignee === "object") {
-          const accountId = (assignee as Raw)["accountId"]
-          if (typeof accountId === "string" && accountId.length > 0) assigneeId = accountId
-        }
-        const labelsRaw = fields["labels"]
-        const labels: Array<string> = []
-        if (Array.isArray(labelsRaw)) {
-          for (const l of labelsRaw) if (typeof l === "string" && l.length > 0) labels.push(l)
-        }
-        const customFields: Record<string, string | null> = {}
-        for (const name of customFieldNames) {
-          const ids = nameToIds.get(name) ?? []
-          let resolved: string | null = null
-          for (const id of ids) {
-            const v = renderCustomFieldValue(fields[id])
-            if (v !== null) {
-              resolved = v
-              break
+        const resObj = asRaw(result)
+        const issues = rawArray(resObj["issues"])
+        for (const issue of issues) {
+          const key = stringOrNull(issue["key"]) ?? ""
+          const fields = asRaw(issue["fields"])
+          const assignee = fields["assignee"]
+          let assigneeId: string | null = null
+          if (isRaw(assignee)) {
+            const accountId = assignee["accountId"]
+            if (typeof accountId === "string" && accountId.length > 0) assigneeId = accountId
+          }
+          const labelsRaw = fields["labels"]
+          const labels: Array<string> = []
+          if (Array.isArray(labelsRaw)) {
+            for (const l of labelsRaw) if (typeof l === "string" && l.length > 0) labels.push(l)
+          }
+          const customFields: Record<string, string | null> = {}
+          for (const name of customFieldNames) {
+            const ids = nameToIds.get(name) ?? []
+            let resolved: string | null = null
+            for (const id of ids) {
+              const v = renderCustomFieldValue(fields[id])
+              if (v !== null) {
+                resolved = v
+                break
+              }
             }
+            customFields[name] = resolved
           }
-          customFields[name] = resolved
+          raws.push({
+            key,
+            summary: stringOrNull(fields["summary"]),
+            assigneeId,
+            labels,
+            customFields
+          })
         }
-        raws.push({
-          key,
-          summary: stringOrNull(fields["summary"]),
-          assigneeId,
-          labels,
-          customFields
-        })
-      }
 
-      const isLast = resObj["isLast"]
-      const next = resObj["nextPageToken"]
-      if (isLast === true || typeof next !== "string" || next.length === 0) break
-      nextPageToken = next
-    }
-
-    const uniqueAssignees = Array.from(
-      new Set(raws.map((t) => t.assigneeId).filter((id): id is string => !!id))
-    )
-    yield* Effect.forEach(uniqueAssignees, (id) => resolveUser(id), { concurrency: 4 })
+        const isLast = resObj["isLast"]
+        const next = resObj["nextPageToken"]
+        if (isLast === true || typeof next !== "string" || next.length === 0) break
+        nextPageToken = next
+      }
 
-    return raws.map((t) => ({
-      key: t.key,
-      summary: t.summary,
-      assignee: t.assigneeId ? userCache.get(t.assigneeId) ?? null : null,
-      labels: t.labels,
-      customFields: t.customFields
-    }))
-  }) as Effect.Effect<ReadonlyArray<VersionTicket>, JiraApiError>)
+      const uniqueAssignees = Array.from(
+        new Set(raws.map((t) => t.assigneeId).filter((id): id is string => !!id))
+      )
+      yield* Effect.forEach(uniqueAssignees, (id) => resolveUser(id), { concurrency: 4 })
+
+      return raws.map((t) => ({
+        key: t.key,
+        summary: t.summary,
+        assignee: t.assigneeId ? userCache.get(t.assigneeId) ?? null : null,
+        labels: t.labels,
+        customFields: t.customFields
+      }))
+    })
 
   const mapVersion = (
     raw: Raw,
@@ -479,7 +479,7 @@ const make = Effect.gen(function*() {
       const name = String(raw["name"] ?? "")
       const driverId = stringOrNull(raw["driver"])
       const declared = extractContributorIds(raw)
-      const approversRaw = (Array.isArray(raw["approvers"]) ? raw["approvers"] : []) as Array<Raw>
+      const approversRaw = rawArray(raw["approvers"])
 
       const tickets = yield* ticketsForVersion(name, customFieldNames, projectKey)
 
@@ -548,38 +548,39 @@ const make = Effect.gen(function*() {
   const listProjectVersions = (
     projectKey: string,
     options?: ListVersionsOptions
-  ): Effect.Effect<ReadonlyArray<Version>, JiraApiError> => (Effect.gen(function*() {
-    const all: Array<Raw> = []
-    let startAt = 0
-    const cap = options?.maxResults
-    const customFieldNames = options?.customFieldNames ?? []
-    for (let page = 0; page < MAX_PAGES; page++) {
-      const result = yield* toEffect(
-        client.v3.client.GET("/rest/api/3/project/{projectIdOrKey}/version", {
-          params: {
-            path: { projectIdOrKey: projectKey },
-            query: { startAt, maxResults: PAGE_SIZE, expand: EXPAND, orderBy: "-releaseDate" }
-          }
-        })
-      ).pipe(
-        Effect.mapError((cause) => new JiraApiError({ message: `Failed to list versions for ${projectKey}`, cause }))
-      )
+  ): Effect.Effect<ReadonlyArray<Version>, JiraApiError> =>
+    Effect.gen(function*() {
+      const all: Array<Raw> = []
+      let startAt = 0
+      const cap = options?.maxResults
+      const customFieldNames = options?.customFieldNames ?? []
+      for (let page = 0; page < MAX_PAGES; page++) {
+        const result = yield* toEffect(
+          client.v3.client.GET("/rest/api/3/project/{projectIdOrKey}/version", {
+            params: {
+              path: { projectIdOrKey: projectKey },
+              query: { startAt, maxResults: PAGE_SIZE, expand: EXPAND, orderBy: "-releaseDate" }
+            }
+          })
+        ).pipe(
+          Effect.mapError((cause) => new JiraApiError({ message: `Failed to list versions for ${projectKey}`, cause }))
+        )
 
-      const resObj = asRaw(result)
-      const values = (Array.isArray(resObj["values"]) ? resObj["values"] : []) as Array<Raw>
-      for (const v of values) {
-        if (options?.released === true && v["released"] !== true) continue
-        if (options?.unreleased === true && v["released"] === true) continue
-        all.push(v)
+        const resObj = asRaw(result)
+        const values = rawArray(resObj["values"])
+        for (const v of values) {
+          if (options?.released === true && v["released"] !== true) continue
+          if (options?.unreleased === true && v["released"] === true) continue
+          all.push(v)
+          if (cap !== undefined && all.length >= cap) break
+        }
         if (cap !== undefined && all.length >= cap) break
+        const isLast = resObj["isLast"]
+        if (isLast === true || values.length < PAGE_SIZE) break
+        startAt += values.length
       }
-      if (cap !== undefined && all.length >= cap) break
-      const isLast = resObj["isLast"]
-      if (isLast === true || values.length < PAGE_SIZE) break
-      startAt += values.length
-    }
-    return yield* Effect.forEach(all, (r) => mapVersion(r, customFieldNames, projectKey), { concurrency: 4 })
-  }) as Effect.Effect<ReadonlyArray<Version>, JiraApiError>)
+      return yield* Effect.forEach(all, (r) => mapVersion(r, customFieldNames, projectKey), { concurrency: 4 })
+    })
 
   const getVersion = (id: string): Effect.Effect<Version, JiraApiError> =>
     toEffect(

package/src/bin.ts

@@ -66,7 +66,7 @@ const program = Effect.gen(function*() {
   )
 }).pipe(
   Effect.provide(NodeStdio.layer),
-  Effect.catchCause((cause) => handleError(cause))
+  Effect.catchCause((cause) => handleError(cause).pipe(Effect.andThen(Effect.failCause(cause))))
 )
 
-NodeRuntime.runMain(program)
+NodeRuntime.runMain(program, { disableErrorReporting: true })

package/src/commands/index.ts

@@ -7,5 +7,6 @@
 export { authCommand } from "./auth.js"
 export { handleError } from "./errorHandler.js"
 export { issueCommand } from "./issue.js"
+export { attachmentCommand } from "./issueAttachment.js"
 export { AppLayer, AuthOnlyLayer, getLayerType, MinimalLayer } from "./layers.js"
 export { versionCommand } from "./version.js"

package/src/commands/issue.ts

@@ -5,9 +5,10 @@
  */
 import { Command } from "effect/unstable/cli"
 import { getCommand } from "./get.js"
+import { attachmentCommand } from "./issueAttachment.js"
 import { searchCommand } from "./search.js"
 
 export const issueCommand = Command.make("issue").pipe(
-  Command.withDescription("Read-only Jira issue commands"),
-  Command.withSubcommands([getCommand, searchCommand])
+  Command.withDescription("Jira issue resource commands"),
+  Command.withSubcommands([attachmentCommand, getCommand, searchCommand])
 )