[WIP] Fix ESLint errors by removing unnecessary type assertions by Copilot · Pull Request #582 · complexdatacollective/Fresco

Copilot · 2026-01-09T10:50:59Z

Thanks for asking me to work on this. I will get started on it and keep this PR's description up to date as I form a plan and make progress.

Original prompt

Problem

The CI workflow is failing due to ESLint errors in lib/interviewer/containers/Interfaces/EgoForm.tsx:
/home/runner/work/Fresco/Fresco/lib/interviewer/containers/Interfaces/EgoForm.tsx
  164:18  error  This assertion is unnecessary since it does not change the type of the expression  @typescript-eslint/no-unnecessary-type-assertion
  165:30  error  This assertion is unnecessary since it does not change the type of the expression  @typescript-eslint/no-unnecessary-type-assertion
Failing job: https://github.com/complexdatacollective/Fresco/actions/runs/20285858847/job/58259246349

Commit with failure: 84fa1ae

Solution

Remove the unnecessary non-null assertion operators (!) from lines 164 and 165 in lib/interviewer/containers/Interfaces/EgoForm.tsx.

Current code (lines 163-166):
<div className="ego-form__introduction">
  <h1>{introductionPanel!.title}</h1>
  <Markdown label={introductionPanel!.text} />
</div>
Updated code:
<div className="ego-form__introduction">
  <h1>{introductionPanel.title}</h1>
  <Markdown label={introductionPanel.text} />
</div>
The non-null assertions are unnecessary because the type system already guarantees that introductionPanel is defined at this point in the code, making the assertions redundant.

This pull request was created from Copilot chat.

💡 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.

Bumps the npm_and_yarn group with 3 updates in the / directory: [brace-expansion](https://github.com/juliangruber/brace-expansion), [js-yaml](https://github.com/nodeca/js-yaml) and [vite](https://github.com/vitejs/vite/tree/HEAD/packages/vite). Updates `brace-expansion` from 1.1.11 to 1.1.12 - [Release notes](https://github.com/juliangruber/brace-expansion/releases) - [Commits](juliangruber/brace-expansion@1.1.11...v1.1.12) Updates `js-yaml` from 3.14.1 to 3.14.2 - [Changelog](https://github.com/nodeca/js-yaml/blob/master/CHANGELOG.md) - [Commits](nodeca/js-yaml@3.14.1...3.14.2) Updates `vite` from 6.3.5 to 7.2.2 - [Release notes](https://github.com/vitejs/vite/releases) - [Changelog](https://github.com/vitejs/vite/blob/main/packages/vite/CHANGELOG.md) - [Commits](https://github.com/vitejs/vite/commits/v7.2.2/packages/vite) --- updated-dependencies: - dependency-name: brace-expansion dependency-version: 1.1.12 dependency-type: indirect dependency-group: npm_and_yarn - dependency-name: js-yaml dependency-version: 3.14.2 dependency-type: indirect dependency-group: npm_and_yarn - dependency-name: vite dependency-version: 7.2.2 dependency-type: indirect dependency-group: npm_and_yarn ... Signed-off-by: dependabot[bot] <support@github.com>

…_yarn/npm_and_yarn-55a67b8840 Bump the npm_and_yarn group across 1 directory with 3 updates

Remove Jest configuration and dependencies in favor of Vitest, which is better integrated with the Next.js ecosystem. Update the test file to use Vitest globals instead of Jest environment configuration.

…anagement

Set node version

Bumps the npm_and_yarn group with 1 update in the / directory: [mdast-util-to-hast](https://github.com/syntax-tree/mdast-util-to-hast). Updates `mdast-util-to-hast` from 13.2.0 to 13.2.1 - [Release notes](https://github.com/syntax-tree/mdast-util-to-hast/releases) - [Commits](syntax-tree/mdast-util-to-hast@13.2.0...13.2.1) --- updated-dependencies: - dependency-name: mdast-util-to-hast dependency-version: 13.2.1 dependency-type: indirect dependency-group: npm_and_yarn ... Signed-off-by: dependabot[bot] <support@github.com>

…_yarn/npm_and_yarn-8f9a9acd57 Bump mdast-util-to-hast from 13.2.0 to 13.2.1 in the npm_and_yarn group across 1 directory

- Triggers on pull requests to main/develop branches - Only builds when relevant source files change - Pushes image to GHCR with pr-<number> and pr-<number>-<sha> tags - Uploads image as artifact for use by other workflows - Adds PR comment with pull/run instructions - Uses build caching for faster subsequent builds

- Remove GHCR registry (no cleanup needed) - Make workflow callable via workflow_call for reuse - Output artifact-name and image-name for consuming workflows - Artifacts auto-delete after 7 days - Use minimal compression for faster artifact transfer

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>

…ons-docker-build-01PXVzGexC1bV2WBthRopZso Create GitHub Actions Docker build workflow

Bumps the npm_and_yarn group with 1 update in the / directory: [next](https://github.com/vercel/next.js). Updates `next` from 14.2.32 to 14.2.35 - [Release notes](https://github.com/vercel/next.js/releases) - [Changelog](https://github.com/vercel/next.js/blob/canary/release.js) - [Commits](vercel/next.js@v14.2.32...v14.2.35) --- updated-dependencies: - dependency-name: next dependency-version: 14.2.35 dependency-type: direct:production dependency-group: npm_and_yarn ... Signed-off-by: dependabot[bot] <support@github.com>

…_yarn/npm_and_yarn-c61ea41056 Bump next from 14.2.32 to 14.2.35 in the npm_and_yarn group across 1 directory

Bumps the npm_and_yarn group with 1 update in the / directory: [storybook](https://github.com/storybookjs/storybook/tree/HEAD/code/core). Updates `storybook` from 9.1.12 to 9.1.17 - [Release notes](https://github.com/storybookjs/storybook/releases) - [Changelog](https://github.com/storybookjs/storybook/blob/next/CHANGELOG.md) - [Commits](https://github.com/storybookjs/storybook/commits/v9.1.17/code/core) --- updated-dependencies: - dependency-name: storybook dependency-version: 9.1.17 dependency-type: direct:development dependency-group: npm_and_yarn ... Signed-off-by: dependabot[bot] <support@github.com>

…_yarn/npm_and_yarn-14e52be3db Bump storybook from 9.1.12 to 9.1.17 in the npm_and_yarn group across 1 directory

Bumps the npm_and_yarn group with 1 update in the / directory: [qs](https://github.com/ljharb/qs). Updates `qs` from 6.14.0 to 6.14.1 - [Changelog](https://github.com/ljharb/qs/blob/main/CHANGELOG.md) - [Commits](ljharb/qs@v6.14.0...v6.14.1) --- updated-dependencies: - dependency-name: qs dependency-version: 6.14.1 dependency-type: indirect dependency-group: npm_and_yarn ... Signed-off-by: dependabot[bot] <support@github.com>

…_yarn/npm_and_yarn-2b901f0e0d Bump qs from 6.14.0 to 6.14.1 in the npm_and_yarn group across 1 directory

vercel · 2026-01-09T10:51:03Z

The latest updates on your projects. Learn more about Vercel for GitHub.

Project	Deployment	Review	Updated (UTC)
fresco-sandbox	Ready	Preview, Comment	Jan 9, 2026 10:55am
fresco-storybook	Ready	Preview, Comment	Jan 9, 2026 10:55am

netlify · 2026-01-09T10:51:05Z

✅ Deploy Preview for fresco-sandbox ready!

Name	Link
🔨 Latest commit	`071465b`
🔍 Latest deploy log	https://app.netlify.com/projects/fresco-sandbox/deploys/6960dd967b99ae00088aafff
😎 Deploy Preview	https://deploy-preview-582--fresco-sandbox.netlify.app
📱 Preview on mobile	Toggle QR Code... Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

netlify · 2026-01-09T10:51:05Z

❌ Deploy Preview for fresco-storybook failed.

Name	Link
🔨 Latest commit	`071465b`
🔍 Latest deploy log	https://app.netlify.com/projects/fresco-storybook/deploys/6960dd97ab7b3600084644d3

jthrilly and others added 22 commits November 17, 2025 12:24

Merge pull request #525 from complexdatacollective/next

3c0717c

Next

make fresco startup crash if DB is not available

8fdd83e

Merge pull request #527 from complexdatacollective/docker-startup

789633f

make fresco startup crash if DB is not available

Merge pull request #533 from complexdatacollective/dependabot/npm_and…

00792dc

…_yarn/npm_and_yarn-55a67b8840 Bump the npm_and_yarn group across 1 directory with 3 updates

Update Node.js to v20.19.0 and migrate from Jest to Vitest

5f4c54f

Remove Jest configuration and dependencies in favor of Vitest, which is better integrated with the Next.js ecosystem. Update the test file to use Vitest globals instead of Jest environment configuration.

chore: Update Node.js and pnpm to latest versions in CI and package m…

e543e82

…anagement

Merge pull request #549 from complexdatacollective/set-node-version

da58eb7

Set node version

Merge pull request #550 from complexdatacollective/dependabot/npm_and…

755f059

…_yarn/npm_and_yarn-8f9a9acd57 Bump mdast-util-to-hast from 13.2.0 to 13.2.1 in the npm_and_yarn group across 1 directory

Update PR docker build to trigger on main and v* branches

1d5bc33

Update .github/workflows/docker-build-pr.yml

f7d72af

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>

Merge pull request #557 from complexdatacollective/claude/github-acti…

af9fc73

…ons-docker-build-01PXVzGexC1bV2WBthRopZso Create GitHub Actions Docker build workflow

Merge pull request #558 from complexdatacollective/dependabot/npm_and…

c8c2641

…_yarn/npm_and_yarn-c61ea41056 Bump next from 14.2.32 to 14.2.35 in the npm_and_yarn group across 1 directory

Merge pull request #571 from complexdatacollective/dependabot/npm_and…

246059b

…_yarn/npm_and_yarn-14e52be3db Bump storybook from 9.1.12 to 9.1.17 in the npm_and_yarn group across 1 directory

Merge pull request #581 from complexdatacollective/dependabot/npm_and…

932a4aa

…_yarn/npm_and_yarn-2b901f0e0d Bump qs from 6.14.0 to 6.14.1 in the npm_and_yarn group across 1 directory

Initial plan

071465b

Copilot AI assigned Copilot and jthrilly Jan 9, 2026

Copilot started work on behalf of jthrilly January 9, 2026 10:51 View session

jthrilly changed the base branch from main to preview-direct-upload January 9, 2026 10:54

vercel Bot deployed to Preview – fresco-sandbox January 9, 2026 10:54 View deployment

jthrilly closed this Jan 9, 2026

vercel Bot deployed to Preview – fresco-storybook January 9, 2026 10:55 View deployment

Copilot AI requested a review from jthrilly January 9, 2026 10:55

Copilot stopped work on behalf of jthrilly due to an error January 9, 2026 10:55
The session was cancelled by the user.

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Uh oh!

Uh oh!

[WIP] Fix ESLint errors by removing unnecessary type assertions#582

[WIP] Fix ESLint errors by removing unnecessary type assertions#582
jthrilly with Copilot wants to merge 22 commits into
preview-direct-uploadfrom
copilot/fix-eslint-errors-in-egoform

Copilot AI commented Jan 9, 2026

Uh oh!

vercel Bot commented Jan 9, 2026 •

edited

Loading

Uh oh!

netlify Bot commented Jan 9, 2026 •

edited

Loading

Uh oh!

netlify Bot commented Jan 9, 2026 •

edited

Loading

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

3 participants

Uh oh!

Uh oh!

Conversation

Copilot AI commented Jan 9, 2026

Problem

Solution

Uh oh!

vercel Bot commented Jan 9, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

netlify Bot commented Jan 9, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

✅ Deploy Preview for fresco-sandbox ready!

Uh oh!

netlify Bot commented Jan 9, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

❌ Deploy Preview for fresco-storybook failed.

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

3 participants

vercel Bot commented Jan 9, 2026 •

edited

Loading

netlify Bot commented Jan 9, 2026 •

edited

Loading

netlify Bot commented Jan 9, 2026 •

edited

Loading