Reduce complexity warnings across scoring logic and UI flows

[Copilot]

Pull Request Template

📝 Summary

This PR removes the complexity warnings called out by npm run lint without changing the story/criteria flow. It breaks up branching-heavy scoring, AI, and UI orchestration paths into smaller helpers and focused subcomponents.

🔧 Changes

• Scoring engine: split complex criteria scoring paths into focused helpers for feedback, suggestions, completeness, per-criterion grading, and shared matching utilities.
• AI service: extracted reusable request/validation helpers in llmService so prompt construction, payload validation, fallback generation, and API handling no longer sit behind large conditional blocks.
• UI flow orchestration: decomposed App into smaller phase/result/panel components and extracted shared helpers for scrolling, history hydration, and AI suggestion application.
• Form rendering: pulled acceptance-criteria and story tooltip/rating logic into dedicated helpers/components to flatten nested render branches while keeping current UX intact.
• Documentation: added targeted JSDoc blocks for the extracted helpers introduced by the refactor.

function buildImproveCriteriaRequest(criteriaDraftInput, criteriaDraftResult, confirmedStory, aiApiKey) {
  return {
    criteria: criteriaDraftInput?.criteria,
    format: criteriaDraftInput?.format,
    draftScore: criteriaDraftResult?.totalScore,
    breakdown: criteriaDraftResult?.breakdown,
    hintTargets: criteriaDraftResult?.hintTargets,
    story: confirmedStory,
    apiKey: aiApiKey,
  };
}

🎯 Motivation

The lint output was dominated by complexity warnings in a few central scoring and UI coordination paths, which made the code harder to change safely. This refactor reduces that branching density while preserving existing behavior and keeping lint/test verification green.

🧪 Testing

• npm run lint
• npm run verify

📸 Screenshots (optional)

N/A

✅ Checklist

• Code compiles and runs
• Tests added or updated (if applicable)
• Documentation updated (if needed)
• No linting or formatting issues
• PR title is clear and descriptive

[Copilot]

Pull request overview

Refactor focused on reducing lint complexity warnings in scoring, AI, and UI orchestration code paths by extracting helpers and decomposing large components. Intent is to keep behavior identical while flattening branching.

Changes:

• Extracted helpers in llmService.js (error normalization, API fetch, validation, fallback builders) and helpers in criteriaScoring.js (feedback/suggestion/completeness/format/testability/specificity/alignment).
• Split App.jsx into many small phase/result/panel/section subcomponents and shared helpers (scrollToPageBottom, buildStoryText, etc.).
• Pulled tooltip data + subcomponents out of StoryForm.jsx, AcceptanceCriteriaForm.jsx, and StoryHistory.jsx.

Reviewed changes

Copilot reviewed 6 out of 6 changed files in this pull request and generated 2 comments.

Show a summary per file

File	Description
src/llmService.js	Refactored error matching, payload validation, fetch, and fallbacks into helpers.
src/criteriaScoring.js	Broke scoring of criteria and single-criterion paths into focused helpers; introduced rule tables.
src/components/StoryHistory.jsx	Extracted StoryDetailsModal subcomponent.
src/components/StoryForm.jsx	Lifted SoThatFeedback and tooltip content tables out of the component body.
src/components/AcceptanceCriteriaForm.jsx	Extracted criterion editor/tooltip subcomponents and tier mapping.
src/App.jsx	Decomposed App into phase/section/panel components and helper functions.