Remove @ts-nocheck from 15 .cjs files and fix type errors

[Copilot]

15 JavaScript files had @ts-nocheck directives disabling TypeScript type checking. All type errors have been resolved and checking is now enabled.

Changes by pattern

Error handling - Replace optional chaining with proper instanceof checks:

// Before
catch (error) {
  const msg = error?.message ?? String(error);
}

// After  
catch (error) {
  const msg = error instanceof Error ? error.message : String(error);
}

Undefined values - Add default values or guards:

const { maxCount = 1 } = config;  // Was: const { maxCount } = config

Null checks - Guard property access on nullable objects:

if (resolved.wasTemporaryId && resolved.resolved) {  // Added && resolved.resolved
  log(resolved.resolved.repo);
}

Type annotations - Add JSDoc where inference fails:

/** @type {Record<string, string>} */
const variables = {};

Dynamic properties - Use @ts-ignore with explanations for intentional dynamic properties on arrays/objects

Files fixed (15)

add_copilot_reviewer.cjs, add_reviewer.cjs, assign_agent_helpers.cjs, assign_copilot_to_created_issues.cjs, assign_issue.cjs, assign_milestone.cjs, assign_to_agent.cjs, assign_to_user.cjs, check_workflow_timestamp_api.cjs, interpolate_prompt.cjs, link_sub_issue.cjs, mcp_http_transport.cjs, notify_comment_error.cjs, parse_copilot_log.cjs, safe_inputs_bootstrap.cjs

Validation

npm run typecheck passes with zero errors.

Original prompt

Find a .cjs with ts-no check and fix type checking.

---

💬 We'd love your input! Share your thoughts on Copilot coding agent in our 2 minute survey.