jsweep: Prioritize files with @ts-nocheck for type checking

[Copilot]

The jsweep workflow now prioritizes removing @ts-nocheck comments from .cjs files to enable TypeScript validation across the codebase (currently 15 files).

Changes

• File selection: Added two-tier priority system

  • Priority 1: Files with @ts-nocheck comments
  • Priority 2: Oldest uncleaned files (fallback)

• Instructions: Added explicit guidance for type checking enablement

  • Remove @ts-nocheck, replace with @ts-check
  • Fix type errors with JSDoc annotations
  • Validate with npm run typecheck before PR

• Constraints: Required typecheck pass for previously unchecked files

• Tests: Validated priority selection and removal instructions in workflow config

Example transformation

// Before: Type checking disabled
// @ts-nocheck
async function processData(data) {
  return data.items.map(item => item.value);
}

// After: Type checking enabled with annotations
// @ts-check
/**
 * @param {{ items: Array<{ value: string }> }} data
 * @returns {Array<string>}
 */
async function processData(data) {
  return data.items.map(item => item.value);
}

Original prompt

Update jsweep to favor fixing .cjs files with "@ts-nocheck" and make them type checked.

Custom agent used: create-agentic-workflow
Design agentic workflows using GitHub Agentic Workflows (gh-aw) extension with interactive guidance on triggers, tools, and security best practices.

---

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