Skip to content

Fix build errors from incomplete import path updates#8

Merged
kertal merged 2 commits into
claude/restructure-app-best-practices-sryEJfrom
copilot/sub-pr-7
Jan 1, 2026
Merged

Fix build errors from incomplete import path updates#8
kertal merged 2 commits into
claude/restructure-app-best-practices-sryEJfrom
copilot/sub-pr-7

Conversation

Copilot AI commented Jan 1, 2026

Copy link
Copy Markdown
Contributor

The consolidation refactor in 6ed1625 missed updating one import path and left types incomplete for assignee fields.

Fixed issues:

  • Updated useLocalStorage.ts import from '../utils/storageUtils''../utils/storage'
  • Added assignee and assignees fields to GitHubEvent.payload.issue type definition
  • Simplified type casting in githubData.ts to use proper types instead of Record<string, unknown>

Type changes:

// Before: type cast workaround
assignee: (payload as Record<string, unknown>).issue?.assignee || null

// After: properly typed
assignee: issue.assignee || null

💡 You can make Copilot smarter by setting up custom instructions, customizing its development environment and configuring Model Context Protocol (MCP) servers. Learn more Copilot coding agent tips in the docs.

Co-authored-by: kertal <463851+kertal@users.noreply.github.com>
@cloudflare-workers-and-pages

cloudflare-workers-and-pages Bot commented Jan 1, 2026

Copy link
Copy Markdown

Deploying git-vegas with  Cloudflare Pages  Cloudflare Pages

Latest commit: 9cb06ce
Status:🚫  Build failed.

View logs

Copilot AI changed the title [WIP] Refactor app structure for improved maintainability Fix build errors from incomplete import path updates Jan 1, 2026
Copilot AI requested a review from kertal January 1, 2026 13:01
@kertal
kertal marked this pull request as ready for review January 1, 2026 14:08
@kertal
kertal merged commit ac5edea into claude/restructure-app-best-practices-sryEJ Jan 1, 2026
1 check failed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants