Remove dead listNoteSlugs() export #10

New issue

Closed

opened 2026-03-14 12:58:41 +00:00 by forgejo_admin · 0 comments

forgejo_admin commented

2026-03-14 12:58:41 +00:00

Owner

Lineage

plan-pal-e-docs → Phase 4 → Phase 4a (dead code cleanup)

Repo

forgejo_admin/pal-e-app

User Story

As a developer
I want dead code removed
So that the codebase stays clean and doesn't confuse future contributors

Context

QA nit from PR #9: listNoteSlugs() in src/lib/api.ts is exported but never imported anywhere. It was replaced by getCachedSlugs() in src/lib/slugCache.ts during the blocker fix.

File Targets

src/lib/api.ts — remove listNoteSlugs() function

Acceptance Criteria

listNoteSlugs() removed from src/lib/api.ts
No remaining imports of listNoteSlugs anywhere
npm run check && npm run build passes

Test Expectations

Run command: npm run check && npm run build

Constraints

Do not change anything else

Checklist

PR opened
Tests pass
No unrelated changes

Closes #9-nit

pal-e-docs — project

### Lineage `plan-pal-e-docs` → Phase 4 → Phase 4a (dead code cleanup) ### Repo `forgejo_admin/pal-e-app` ### User Story As a developer I want dead code removed So that the codebase stays clean and doesn't confuse future contributors ### Context QA nit from PR #9: `listNoteSlugs()` in `src/lib/api.ts` is exported but never imported anywhere. It was replaced by `getCachedSlugs()` in `src/lib/slugCache.ts` during the blocker fix. ### File Targets - `src/lib/api.ts` — remove `listNoteSlugs()` function ### Acceptance Criteria - [ ] `listNoteSlugs()` removed from `src/lib/api.ts` - [ ] No remaining imports of `listNoteSlugs` anywhere - [ ] `npm run check && npm run build` passes ### Test Expectations - Run command: `npm run check && npm run build` ### Constraints - Do not change anything else ### Checklist - [ ] PR opened - [ ] Tests pass - [ ] No unrelated changes Closes #9-nit ### Related - `pal-e-docs` — project