Fix lint and TypeScript errors blocking CI pipeline #94

New issue

Closed

opened 2026-03-28 22:50:36 +00:00 by forgejo_admin · 0 comments

forgejo_admin commented

2026-03-28 22:50:36 +00:00

Contributor

Type

Feature

Lineage

Related to forgejo_admin/pal-e-platform #234 (parent issue tracking ImagePullBackOff).

Repo

forgejo_admin/pal-e-docs-app

User Story

As a developer
I want the CI check+lint stage to pass
So that the Docker image can be built and the pod stops being in ImagePullBackOff

Context

The Woodpecker CI pipeline (#110) fails at the check+lint stage. No Docker image has ever been built for pal-e-docs-app because TypeScript and ESLint errors block the pipeline. The pod is stuck in ImagePullBackOff because Harbor has no image to pull.

Two TypeScript errors (implicit any on event parameters) and six ESLint errors (unused imports, Set vs SvelteSet, unused svelte-ignore comment, unused variable) need to be fixed.

File Targets

Files the agent should modify:

src/routes/boards/[slug]/+page.svelte -- type the e parameter on lines 826 and 829
src/routes/+layout.svelte -- remove unused checkAuth import alias
src/routes/graph/+page.svelte -- use SvelteSet on lines 91 and 136, remove unused svelte-ignore on line 341
src/routes/notes/+page.svelte -- use SvelteSet on line 39
src/routes/search/+page.svelte -- remove unused searchError variable

Files the agent should NOT touch:

All other files -- scope is limited to the 8 reported errors

Acceptance Criteria

npm run check passes with 0 errors
npm run lint passes with 0 errors
No functional changes to application behavior

Test Expectations

Run command: npm run check && npm run lint
No unit tests needed -- these are type annotation and lint fixes only

Constraints

Fix ONLY the reported errors. No refactoring.
Do NOT add new dependencies.
Use SvelteSet from svelte/reactivity where the lint rule requires it.

Checklist

PR opened
npm run check passes
npm run lint passes
No unrelated changes

pal-e-docs -- project this affects
forgejo_admin/pal-e-platform #234 -- parent issue

### Type Feature ### Lineage Related to `forgejo_admin/pal-e-platform #234` (parent issue tracking ImagePullBackOff). ### Repo `forgejo_admin/pal-e-docs-app` ### User Story As a developer I want the CI check+lint stage to pass So that the Docker image can be built and the pod stops being in ImagePullBackOff ### Context The Woodpecker CI pipeline (#110) fails at the check+lint stage. No Docker image has ever been built for pal-e-docs-app because TypeScript and ESLint errors block the pipeline. The pod is stuck in ImagePullBackOff because Harbor has no image to pull. Two TypeScript errors (implicit `any` on event parameters) and six ESLint errors (unused imports, `Set` vs `SvelteSet`, unused svelte-ignore comment, unused variable) need to be fixed. ### File Targets Files the agent should modify: - `src/routes/boards/[slug]/+page.svelte` -- type the `e` parameter on lines 826 and 829 - `src/routes/+layout.svelte` -- remove unused `checkAuth` import alias - `src/routes/graph/+page.svelte` -- use SvelteSet on lines 91 and 136, remove unused svelte-ignore on line 341 - `src/routes/notes/+page.svelte` -- use SvelteSet on line 39 - `src/routes/search/+page.svelte` -- remove unused `searchError` variable Files the agent should NOT touch: - All other files -- scope is limited to the 8 reported errors ### Acceptance Criteria - [ ] `npm run check` passes with 0 errors - [ ] `npm run lint` passes with 0 errors - [ ] No functional changes to application behavior ### Test Expectations - Run command: `npm run check && npm run lint` - No unit tests needed -- these are type annotation and lint fixes only ### Constraints - Fix ONLY the reported errors. No refactoring. - Do NOT add new dependencies. - Use `SvelteSet` from `svelte/reactivity` where the lint rule requires it. ### Checklist - [ ] PR opened - [ ] `npm run check` passes - [ ] `npm run lint` passes - [ ] No unrelated changes ### Related - `pal-e-docs` -- project this affects - `forgejo_admin/pal-e-platform #234` -- parent issue