opensoma 0.5.1 → 0.6.0

package/src/formatters.ts

@@ -8,12 +8,12 @@ import {
   ApprovalListItemSchema,
   type Dashboard,
   DashboardSchema,
-  type EventListItem,
-  EventListItemSchema,
   type MemberInfo,
   MemberInfoSchema,
   type MentoringDetail,
   MentoringDetailSchema,
+  type MentoringEditForm,
+  MentoringEditFormSchema,
   type MentoringListItem,
   MentoringListItemSchema,
   type NoticeDetail,
@@ -28,6 +28,13 @@ import {
   ReportListItemSchema,
   type RoomCard,
   RoomCardSchema,
+  type RoomReservationDetail,
+  RoomReservationDetailSchema,
+  type RoomReservationListItem,
+  RoomReservationListItemSchema,
+  type RoomReservationStatus,
+  type ScheduleListItem,
+  ScheduleListItemSchema,
   type TeamInfo,
   TeamInfoSchema,
 } from './types'
@@ -105,6 +112,83 @@ export function parseMentoringDetail(html: string, id = 0): MentoringDetail {
   })
 }
 
+export function parseMentoringEditForm(html: string, id = 0): MentoringEditForm {
+  const root = parse(html)
+  const form = root.querySelector('form#board') ?? root.querySelector('form')
+  const fields = readFormFields(form)
+  const initial = parseInitialDataBlock(html)
+
+  const pick = (key: string): string => {
+    const formValue = fields[key]
+    if (formValue && formValue.trim() !== '') return formValue
+    return initial[key] ?? ''
+  }
+  const receiptType = pick('receiptType') === 'DIRECT' ? 'DIRECT' : 'UNTIL_LECTURE'
+
+  return MentoringEditFormSchema.parse({
+    id: id || extractNumber(fields.qustnrSn ?? ''),
+    title: fields.qustnrSj ?? '',
+    reportCd: pick('reportCd'),
+    receiptType,
+    bgndeDate: pick('bgndeDate'),
+    bgndeTime: pick('bgndeTime'),
+    enddeDate: pick('enddeDate'),
+    enddeTime: pick('enddeTime'),
+    eventDt: pick('eventDt'),
+    eventStime: pick('eventStime'),
+    eventEtime: pick('eventEtime'),
+    applyCnt: extractNumber(fields.applyCnt ?? ''),
+    place: fields.place ?? '',
+  })
+}
+
+function parseInitialDataBlock(html: string): Record<string, string> {
+  const block = html.match(/var\s+INITIAL_DATA\s*=\s*\{([\s\S]*?)\};/)?.[1]
+  if (!block) return {}
+  const fields: Record<string, string> = {}
+  for (const match of block.matchAll(/(\w+)\s*:\s*'([^']*)'/g)) {
+    fields[match[1]] = match[2]
+  }
+  return fields
+}
+
+function readFormFields(form: HTMLElement | null): Record<string, string> {
+  if (!form) return {}
+  const fields: Record<string, string> = {}
+
+  for (const input of form.querySelectorAll('input')) {
+    const name = input.getAttribute('name')
+    if (!name) continue
+    const type = (input.getAttribute('type') ?? '').toLowerCase()
+    if (type === 'radio' || type === 'checkbox') {
+      const rawAttrs = input.rawAttrs ?? ''
+      const isChecked = /\bchecked\b/.test(rawAttrs)
+      if (isChecked) {
+        fields[name] = input.getAttribute('value') ?? ''
+      } else if (!(name in fields)) {
+        fields[name] = ''
+      }
+    } else {
+      fields[name] = input.getAttribute('value') ?? ''
+    }
+  }
+
+  for (const select of form.querySelectorAll('select')) {
+    const name = select.getAttribute('name')
+    if (!name) continue
+    const selected = select.querySelector('option[selected]')
+    fields[name] = selected?.getAttribute('value') ?? ''
+  }
+
+  for (const textarea of form.querySelectorAll('textarea')) {
+    const name = textarea.getAttribute('name')
+    if (!name) continue
+    fields[name] = textarea.text ?? ''
+  }
+
+  return fields
+}
+
 export function parseRoomList(html: string): RoomCard[] {
   const root = parse(html)
   const cards = root.querySelectorAll('ul.bbs-reserve > li.item')
@@ -135,6 +219,86 @@ export function parseRoomSlots(html: string): RoomCard['timeSlots'] {
   return slots.map(parseRoomTimeSlot).filter((slot) => Boolean(slot.time))
 }
 
+export function parseRoomReservationDetail(html: string): RoomReservationDetail {
+  const root = parse(html)
+  const form = root.querySelector('form#frm') ?? root.querySelector('form')
+  const fields: Record<string, string> = {}
+  for (const input of form?.querySelectorAll('input, select, textarea') ?? []) {
+    const name = input.getAttribute('name')
+    if (!name) continue
+    fields[name] = input.getAttribute('value') ?? input.text ?? ''
+  }
+
+  const statusCode = fields.receiptStatCd ?? ''
+
+  return RoomReservationDetailSchema.parse({
+    rentId: extractNumber(fields.rentId ?? ''),
+    itemId: extractNumber(fields.itemId ?? ''),
+    title: fields.title ?? '',
+    date: fields.rentDt ?? extractReservationDate(fields.rentBgnde),
+    startTime: extractReservationTime(fields.rentBgnde),
+    endTime: extractReservationTime(fields.rentEndde),
+    attendees: extractNumber(fields.rentNum ?? '') || 1,
+    notes: fields.infoCn ?? '',
+    status: resolveReservationStatus(statusCode),
+    statusCode,
+  })
+}
+
+function extractReservationDate(value: string | undefined): string {
+  if (!value) return ''
+  return value.slice(0, 10)
+}
+
+function extractReservationTime(value: string | undefined): string {
+  if (!value) return ''
+  const match = value.match(/\d{2}:\d{2}/)
+  return match?.[0] ?? ''
+}
+
+function resolveReservationStatus(code: string): RoomReservationStatus {
+  if (code === 'RS001') return 'confirmed'
+  if (code === 'RS002') return 'cancelled'
+  return 'unknown'
+}
+
+export function parseRoomReservationList(html: string): RoomReservationListItem[] {
+  return findTableRows(html, 7).map((cells) => {
+    const venueLink = cells[1]?.querySelector('a')
+    const rentId = extractUrlParam(venueLink?.getAttribute('href'), 'rentId')
+    const titleLink = cells[2]?.querySelector('.rel a')
+    const periodText = cleanText(cells[3])
+    const statusLabel = cleanText(cells[5])
+
+    return RoomReservationListItemSchema.parse({
+      rentId,
+      venue: cleanText(venueLink ?? cells[1]),
+      title: cleanText(titleLink),
+      date: extractFirstDate(periodText),
+      startTime: extractReservationStartTime(periodText),
+      endTime: extractReservationEndTime(periodText),
+      author: cleanText(cells[4]),
+      status: statusLabelToStatus(statusLabel),
+      statusLabel,
+      registeredAt: cleanText(cells[6]),
+    })
+  })
+}
+
+function statusLabelToStatus(label: string): RoomReservationStatus {
+  if (label.includes('예약완료')) return 'confirmed'
+  if (label.includes('예약취소') || label.includes('취소')) return 'cancelled'
+  return 'unknown'
+}
+
+function extractReservationStartTime(text: string): string {
+  return text.match(/\d{2}:\d{2}/)?.[0] ?? ''
+}
+
+function extractReservationEndTime(text: string): string {
+  return text.match(/\d{2}:\d{2}/g)?.[1] ?? ''
+}
+
 export function parseDashboard(html: string): Dashboard {
   const root = parse(html)
   const profileCard = root.querySelector('ul.dash-top > li.dash-card')
@@ -153,6 +317,7 @@ export function parseDashboard(html: string): Dashboard {
     organization: extractDashEtcValue(dashEtc, '소속'),
     position: extractDashEtcValue(dashEtc, '직책'),
     team: team.name || team.members || team.mentor ? team : undefined,
+    teams: [],
     mentoringSessions: parseDashboardLinks(root, (href) => href.includes('/mentoLec/')),
     roomReservations: parseDashboardLinks(root, (href) => href.includes('/itemRent/') || href.includes('/officeMng/')),
   })
@@ -198,16 +363,12 @@ export function parseTeamInfo(html: string): TeamInfo {
   const root = parse(html)
   const cards = root.querySelectorAll('ul.bbs-team > li')
   const summaryText = cleanText(root.querySelector('p.ico-team'))
-  const summaryNumbers = summaryText.match(/(\d+)\/(\d+)팀/)
+  const summary = parseTeamSummary(summaryText)
 
   return TeamInfoSchema.parse({
-    teams: cards.map((card) => ({
-      name: cleanText(card.querySelector('.top strong.t a')),
-      memberCount: extractMemberCount(cleanText(card)),
-      joinStatus: extractJoinStatus(card),
-    })),
-    currentTeams: summaryNumbers ? Number.parseInt(summaryNumbers[1], 10) : 0,
-    maxTeams: summaryNumbers ? Number.parseInt(summaryNumbers[2], 10) : 0,
+    teams: cards.map((card) => parseTeamCard(card)),
+    currentTeams: summary.current,
+    maxTeams: summary.max,
   })
 }
 
@@ -225,26 +386,42 @@ export function parseMemberInfo(html: string): MemberInfo {
   })
 }
 
-export function parseEventList(html: string): EventListItem[] {
-  return findTableRows(html, 7).map((cells) =>
-    EventListItemSchema.parse({
-      id: extractLinkParam(cells[2], ['qustnrSn', 'bbsId', 'nttId']) || extractNumber(cleanText(cells[0])),
-      category: cleanText(cells[1]),
-      title: cleanText(cells[2]?.querySelector('a') ?? cells[2]),
-      registrationPeriod: extractDateRange(cleanText(cells[3])),
-      eventPeriod: extractDateRange(cleanText(cells[4])),
-      status: stripWrappingBrackets(cleanText(cells[5])),
-      createdAt: cleanText(cells[6]),
-    }),
-  )
+export function parseScheduleList(html: string): { items: ScheduleListItem[]; pagination: Pagination } {
+  const root = parse(html)
+  const monthlyScheduleTable = findTableByHeaders(root, ['날짜', '구분', '제목'])
+  const scheduleRows = monthlyScheduleTable?.querySelectorAll('tbody tr') ?? []
+  const items = scheduleRows
+    .map((row) => row.querySelectorAll('td'))
+    .filter((cells) => cells.length === 3)
+    .map((cells, index) =>
+      ScheduleListItemSchema.parse({
+        id: index + 1,
+        category: cleanText(cells[1]),
+        title: cleanText(cells[2]),
+        period: extractDateRange(cleanText(cells[0])),
+      }),
+    )
+
+  // SWMaestro's monthly schedule page does not render the bbs-total pagination block,
+  // so synthesize a single-page response when items exist but parsePagination found none.
+  const parsedPagination = parsePagination(html, { itemCount: items.length })
+  const pagination =
+    parsedPagination.total === 0 && items.length > 0
+      ? PaginationSchema.parse({ total: items.length, currentPage: 1, totalPages: 1 })
+      : parsedPagination
+
+  return { items, pagination }
 }
 
 export function parseApplicationHistory(html: string): ApplicationHistoryItem[] {
-  return findTableRows(html, 10).map((cells) =>
-    ApplicationHistoryItemSchema.parse({
+  return findTableRows(html, 10).map((cells) => {
+    const link = cells[2]?.querySelector('a')
+    const url = link?.getAttribute('href')
+    return ApplicationHistoryItemSchema.parse({
       id: extractNumber(cleanText(cells[0])),
       category: cleanText(cells[1]),
-      title: cleanText(cells[2]?.querySelector('a') ?? cells[2]),
+      title: cleanText(link ?? cells[2]),
+      ...(url ? { url } : {}),
       author: cleanText(cells[3]),
       sessionDate: normalizeDate(cleanText(cells[4])),
       appliedAt: cleanText(cells[5]),
@@ -252,23 +429,46 @@ export function parseApplicationHistory(html: string): ApplicationHistoryItem[]
       approvalStatus: stripWrappingBrackets(cleanText(cells[7])),
       applicationDetail: cleanText(cells[8]),
       note: cleanText(cells[9]),
-    }),
-  )
+    })
+  })
 }
 
-export function parsePagination(html: string): Pagination {
+export function parsePagination(html: string, options?: { itemCount?: number }): Pagination {
   const root = parse(html)
   const items = root.querySelectorAll('ul.bbs-total > li').map((item) => cleanText(item))
   const total = extractNumber(items.find((item) => item.includes('Total')) ?? '')
   const pageMatch = items.find((item) => item.includes('Page'))?.match(/(\d+)\s*\/\s*(\d+)\s*Page/i)
+  const currentPage = pageMatch ? Number.parseInt(pageMatch[1], 10) : 1
+  const parsedTotalPages = pageMatch ? Number.parseInt(pageMatch[2], 10) : 1
 
   return PaginationSchema.parse({
     total,
-    currentPage: pageMatch ? Number.parseInt(pageMatch[1], 10) : 1,
-    totalPages: pageMatch ? Number.parseInt(pageMatch[2], 10) : 1,
+    currentPage,
+    totalPages: deriveTotalPages({ total, currentPage, parsedTotalPages, itemCount: options?.itemCount }),
   })
 }
 
+function deriveTotalPages({
+  total,
+  currentPage,
+  parsedTotalPages,
+  itemCount,
+}: {
+  total: number
+  currentPage: number
+  parsedTotalPages: number
+  itemCount: number | undefined
+}): number {
+  if (total <= 0) return 1
+  const fallback = Math.max(1, parsedTotalPages, currentPage)
+  if (itemCount === undefined) return fallback
+  // SWMaestro's `1/50 Page` text is unreliable on some lists (e.g. report) — the server emits a
+  // hardcoded cap regardless of actual data. When the visible items already cover the total,
+  // there is only one page; correct the bogus value rather than echo it.
+  if (currentPage === 1 && total <= itemCount) return 1
+  return fallback
+}
+
 export function parseCsrfToken(html: string): string {
   const token = parse(html).querySelector('input[name="csrfToken"]')?.getAttribute('value')
 
@@ -281,12 +481,16 @@ export function parseCsrfToken(html: string): string {
 
 export function parseReportList(html: string): ReportListItem[] {
   return findTableRows(html, 9).map((cells) => {
-    const link = cells[2]?.querySelector('a')
+    // Native list cell layout: `td.tit > div.date_m > span.l > a` holds the
+    // category, and `td.tit > div.rel > a` holds the actual title that matches
+    // the `제목` label on the detail page (e.g. "[자유 멘토링] 2026년 04월 30일
+    // 멘토링 보고"). Prefer the `.rel a` so list and detail titles agree.
+    const titleLink = cells[2]?.querySelector('div.rel a') ?? cells[2]?.querySelector('a')
 
     return ReportListItemSchema.parse({
-      id: extractUrlParam(link?.getAttribute('href'), 'reportId'),
+      id: extractUrlParam(titleLink?.getAttribute('href'), 'reportId'),
       category: cleanText(cells[1]),
-      title: cleanText(link ?? cells[2]),
+      title: cleanText(titleLink ?? cells[2]),
       progressDate: cleanText(cells[3]),
       status: cleanText(cells[4]),
       author: cleanText(cells[5]),
@@ -399,6 +603,14 @@ function findTableRows(html: string, cellCount: number): HTMLElement[][] {
     .filter((cells) => cells.length === cellCount)
 }
 
+function findTableByHeaders(root: HTMLElement, headers: string[]): HTMLElement | undefined {
+  return root.querySelectorAll('table').find((table) => {
+    const tableHeaders = table.querySelectorAll('thead th').map((header) => cleanText(header))
+
+    return headers.every((header, index) => tableHeaders[index] === header)
+  })
+}
+
 function extractLabelMap(root: HTMLElement): LabelMap {
   const map: LabelMap = {}
 
@@ -560,19 +772,6 @@ function extractUrlParam(url: string | undefined, key: string): number {
   }
 }
 
-function extractLinkParam(cell: HTMLElement | undefined, keys: string[]): number {
-  const href = cell?.querySelector('a')?.getAttribute('href')
-
-  for (const key of keys) {
-    const value = extractUrlParam(href, key)
-    if (value > 0) {
-      return value
-    }
-  }
-
-  return 0
-}
-
 function extractDateRange(text: string): { start: string; end: string } {
   const dates = text.match(/\d{4}[.-]\d{2}[.-]\d{2}/g)?.map(normalizeDate) ?? []
   return { start: dates[0] ?? '', end: dates[1] ?? '' }
@@ -605,18 +804,73 @@ function extractCapacity(text: string): number {
   return match ? Number.parseInt(match[1], 10) : 0
 }
 
-function extractMemberCount(text: string): number {
-  const match = text.match(/(\d+)\s*명/)
-  return match ? Number.parseInt(match[1], 10) : 0
+function parseTeamCard(card: HTMLElement) {
+  const titleAnchor = card.querySelector('.top strong.t a')
+  const teamPageGoArgs = parseTeamPageGoArgs(titleAnchor?.getAttribute('onclick') ?? '')
+  const infoItems = card.querySelectorAll('.top ul.info > li')
+  const ictItems = card.querySelectorAll('.bot ul.ict > li')
+
+  return {
+    name: cleanText(titleAnchor),
+    projectName: cleanText(card.querySelector('.top .add-txt')),
+    ownerId: teamPageGoArgs[1] ?? '',
+    teamId: teamPageGoArgs[2] ?? '',
+    leader: extractInfoLeader(infoItems),
+    members: extractInfoMembers(infoItems, '팀원'),
+    mentors: extractInfoMembers(infoItems, '멘토'),
+    ictCategoryMajor: extractIctCategory(ictItems, '대'),
+    ictCategoryMinor: extractIctCategory(ictItems, '중'),
+    teamCompleted: card.querySelector('.team-com .t1') !== null,
+    mentorCompleted: card.querySelector('.team-com .t2') !== null,
+    joinStatus: extractJoinStatus(card),
+  }
+}
+
+function parseTeamSummary(text: string): { current: number; max: number } {
+  const match = text.match(/(\d+)\s*(?:\/\s*(\d+))?\s*팀/)
+  return {
+    current: match ? Number.parseInt(match[1], 10) : 0,
+    max: match?.[2] ? Number.parseInt(match[2], 10) : 0,
+  }
+}
+
+function parseTeamPageGoArgs(onclick: string): string[] {
+  // teamPageGo('전수열','f6d192...','ef9d3b...');
+  const match = onclick.match(/teamPageGo\(\s*'([^']*)'\s*,\s*'([^']*)'\s*,\s*'([^']*)'\s*\)/)
+  return match ? [match[1], match[2], match[3]] : []
+}
+
+function findInfoItemByLabel(items: HTMLElement[], label: string): HTMLElement | undefined {
+  return items.find((item) => cleanText(item.querySelector('strong')).startsWith(label))
+}
+
+function extractInfoLeader(items: HTMLElement[]): string {
+  const leaderItem = findInfoItemByLabel(items, '팀장')
+  return cleanText(leaderItem?.querySelector('span a')) || cleanText(leaderItem?.querySelector('span'))
+}
+
+function extractInfoMembers(items: HTMLElement[], label: string): { name: string; userId: string }[] {
+  const item = findInfoItemByLabel(items, label)
+  if (!item) return []
+  return item.querySelectorAll('span a').map((anchor) => ({
+    name: cleanText(anchor),
+    userId: extractPopuserUserId(anchor.getAttribute('href') ?? ''),
+  }))
+}
+
+function extractPopuserUserId(href: string): string {
+  // javascript: popuser('113889210806408397ea7db2ea855f71')
+  const match = href.match(/popuser\(\s*'([^']*)'/)
+  return match ? match[1] : ''
+}
+
+function extractIctCategory(items: HTMLElement[], group: '대' | '중'): string {
+  const item = items.find((node) => cleanText(node).includes(`(${group})`))
+  return cleanText(item?.querySelector('span'))
 }
 
 function extractJoinStatus(card: HTMLElement): string {
-  return (
-    cleanText(card.querySelector('button')) ||
-    cleanText(card.querySelector('.btn')) ||
-    cleanText(card.querySelector('[class*="join"]')) ||
-    ''
-  )
+  return cleanText(card.querySelector('.bot .btn_w .btn-team'))
 }
 
 function extractTrailingStatus(text: string): string {

package/src/http.test.ts

@@ -2,7 +2,7 @@ import { afterEach, describe, expect, it, mock } from 'bun:test'
 
 import { MENU_NO } from './constants'
 import { AuthenticationError } from './errors'
-import { SomaHttp } from './http'
+import { SomaHttp, UserGb } from './http'
 
 const originalFetch = globalThis.fetch
 
@@ -81,6 +81,46 @@ describe('SomaHttp', () => {
     await expect(http.post('/mypage/test.do', {})).rejects.toThrow('잘못된 접근입니다.')
   })
 
+  it('treats success-indicator alerts (정상적으로 등록하였습니다) as non-errors', async () => {
+    const fetchMock = mock(async () =>
+      createResponse(
+        `<html><body><script type='text/javascript'>alert('정상적으로 등록하였습니다.');location.href='/sw/mypage/itemRent/list.do';</script></body></html>`,
+      ),
+    )
+    globalThis.fetch = fetchMock as typeof fetch
+
+    const http = new SomaHttp({ sessionCookie: 'session-1', csrfToken: 'csrf-1' })
+
+    const html = await http.post('/mypage/itemRent/insert.do', {})
+    expect(html).toContain('정상적으로 등록하였습니다')
+  })
+
+  it('treats success-indicator alerts (완료되었습니다) as non-errors', async () => {
+    const fetchMock = mock(async () =>
+      createResponse(
+        `<html><body><script>alert('예약이 완료되었습니다.');location.href='/sw/mypage/itemRent/list.do';</script></body></html>`,
+      ),
+    )
+    globalThis.fetch = fetchMock as typeof fetch
+
+    const http = new SomaHttp({ sessionCookie: 'session-1', csrfToken: 'csrf-1' })
+
+    await expect(http.post('/mypage/itemRent/insert.do', {})).resolves.toBeDefined()
+  })
+
+  it('still surfaces non-success alerts when followed by location.href', async () => {
+    const fetchMock = mock(async () =>
+      createResponse(
+        `<html><body><script>alert('이미 예약된 시간입니다.');location.href='/sw/mypage/itemRent/list.do';</script></body></html>`,
+      ),
+    )
+    globalThis.fetch = fetchMock as typeof fetch
+
+    const http = new SomaHttp({ sessionCookie: 'session-1', csrfToken: 'csrf-1' })
+
+    await expect(http.post('/mypage/itemRent/insert.do', {})).rejects.toThrow('이미 예약된 시간입니다.')
+  })
+
   it('ignores alert() calls nested inside function bodies (form validation scripts)', async () => {
     const pageWithValidationScript = `<html><head><title>AI·SW마에스트로 서울</title></head><body>
       <ul class="bbs-reserve"><li class="item">room data</li></ul>
@@ -314,7 +354,12 @@ describe('SomaHttp', () => {
       createResponse(
         JSON.stringify({
           resultCode: 'fail',
-          userVO: { userId: 'user@example.com', userNm: 'Test' },
+          userVO: {
+            userId: 'user@example.com',
+            userNm: 'Test',
+            userNo: 'a1b2c3',
+            userGb: 'T',
+          },
         }),
         [],
         'application/json',
@@ -325,6 +370,8 @@ describe('SomaHttp', () => {
     await expect(new SomaHttp().checkLogin()).resolves.toEqual({
       userId: 'user@example.com',
       userNm: 'Test',
+      userNo: 'a1b2c3',
+      userGb: UserGb.Mentor,
     })
     expect(loggedInMock).toHaveBeenCalledWith('https://www.swmaestro.ai/sw/member/user/checkLogin.json', {
       method: 'GET',
@@ -345,6 +392,28 @@ describe('SomaHttp', () => {
     await expect(new SomaHttp().checkLogin()).resolves.toBeNull()
   })
 
+  it('maps trainee userGb responses to the UserGb enum', async () => {
+    globalThis.fetch = mock(async () =>
+      createResponse(
+        JSON.stringify({
+          resultCode: 'fail',
+          userVO: {
+            userId: 'trainee@example.com',
+            userNm: 'Trainee',
+            userNo: 'u-1',
+            userGb: 'C',
+          },
+        }),
+        [],
+        'application/json',
+      ),
+    ) as typeof fetch
+
+    await expect(new SomaHttp().checkLogin()).resolves.toMatchObject({
+      userGb: UserGb.Trainee,
+    })
+  })
+
   it('returns null from checkLogin when the server redirects to the login page', async () => {
     const fetchMock = mock(async () =>
       createResponse('', [], 'text/html', {

package/src/http.ts

@@ -15,12 +15,26 @@ interface RequestOptions {
 
 interface CheckLoginResponse {
   resultCode?: string
-  userVO?: { userId?: string; userNm?: string; userSn?: number }
+  userVO?: {
+    userId?: string
+    userNm?: string
+    userSn?: number
+    userNo?: string
+    userGb?: string
+  }
 }
 
 export interface UserIdentity {
   userId: string
   userNm: string
+  userNo: string
+  userGb: UserGb
+}
+
+export enum UserGb {
+  Unknown = '',
+  Trainee = 'C',
+  Mentor = 'T',
 }
 
 interface HeadersWithCookieHelpers extends Omit<Headers, 'getSetCookie'> {
@@ -231,6 +245,17 @@ export class SomaHttp {
     return message.includes('세션') && /초기화|만료|유효하지/.test(message)
   }
 
+  private isSuccessAlert(message: string): boolean {
+    // SWMaestro returns alert() scripts for both errors and successes (e.g. after room
+    // reservation the server responds with alert('정상적으로 등록하였습니다.');location.href=...).
+    // These success alerts must not be surfaced as errors to callers. The server emits
+    // variants with or without whitespace between the verb and 되었습니다 (e.g. "수정 되었습니다"
+    // vs "수정되었습니다"), so match both spellings.
+    return /정상적으로|등록\s?하였습니다|등록\s?되었습니다|수정\s?되었습니다|저장\s?되었습니다|완료\s?되었습니다|삭제\s?되었습니다/.test(
+      message,
+    )
+  }
+
   private extractErrorFromResponse(body: string, location: string | null, path?: string): string | null {
     this.log(
       'extractErrorFromResponse',
@@ -250,6 +275,10 @@ export class SomaHttp {
       if (this.isSessionExpiredError(alertMatch[1])) {
         return '__AUTH_ERROR__'
       }
+      if (this.isSuccessAlert(alertMatch[1])) {
+        this.log('Alert is a success message, ignoring:', alertMatch[1])
+        return null
+      }
       return alertMatch[1]
     }
 
@@ -408,7 +437,12 @@ export class SomaHttp {
 
     const userId = json.userVO?.userId
     if (!userId) return null
-    return { userId, userNm: json.userVO?.userNm ?? '' }
+    return {
+      userId,
+      userNm: json.userVO?.userNm ?? '',
+      userNo: json.userVO?.userNo ?? '',
+      userGb: parseUserGb(json.userVO?.userGb),
+    }
   }
 
   async logout(): Promise<void> {
@@ -532,3 +566,8 @@ export class SomaHttp {
     }
   }
 }
+
+function parseUserGb(value: string | undefined): UserGb {
+  if (value === UserGb.Trainee || value === UserGb.Mentor) return value
+  return UserGb.Unknown
+}

package/src/index.ts

@@ -1,7 +1,16 @@
+export { AgentBrowserLauncher, assertSwmaestroUrl, buildStorageState } from './agent-browser-launcher'
+export type {
+  AgentBrowserLauncherOptions,
+  AgentBrowserLaunchInput,
+  LaunchResult,
+  Spawner,
+  SpawnedProcess,
+} from './agent-browser-launcher'
 export { SomaClient } from './client'
 export type { SomaClientOptions } from './client'
 export { AuthenticationError } from './errors'
-export { SomaHttp } from './http'
+export { SomaHttp, UserGb } from './http'
+export type { UserIdentity } from './http'
 export { CredentialManager } from './credential-manager'
 export { TozHttp } from './toz-http'
 export type { TozHttpOptions, TozHttpState } from './toz-http'