Remove dead listNoteSlugs() export #11

Merged

forgejo_admin merged 1 commit from 10-remove-dead-listnoteslugs into main

2026-03-14 13:15:44 +00:00

forgejo_admin commented

2026-03-14 13:00:06 +00:00

Owner

Summary

Remove the unused listNoteSlugs() function from src/lib/api.ts. It was replaced by the slug cache in src/lib/slugCache.ts and has zero remaining import references.

Changes

src/lib/api.ts -- deleted the listNoteSlugs() function (lines 226-229), a 4-line wrapper that fetched all notes just to extract slugs. The slug cache handles this with TTL and deduplication.

Test Plan

npm run check passes with 0 errors, 0 warnings
npm run build succeeds
grep -r listNoteSlugs src/ confirms zero remaining references

Review Checklist

No remaining imports or usages of listNoteSlugs in the codebase
npm run check clean
npm run build succeeds
slugCache.ts uses listNotes directly, unaffected by this removal

Forgejo issue: #10

Closes #10

## Summary Remove the unused `listNoteSlugs()` function from `src/lib/api.ts`. It was replaced by the slug cache in `src/lib/slugCache.ts` and has zero remaining import references. ## Changes - **src/lib/api.ts** -- deleted the `listNoteSlugs()` function (lines 226-229), a 4-line wrapper that fetched all notes just to extract slugs. The slug cache handles this with TTL and deduplication. ## Test Plan - `npm run check` passes with 0 errors, 0 warnings - `npm run build` succeeds - `grep -r listNoteSlugs src/` confirms zero remaining references ## Review Checklist - [x] No remaining imports or usages of `listNoteSlugs` in the codebase - [x] `npm run check` clean - [x] `npm run build` succeeds - [x] `slugCache.ts` uses `listNotes` directly, unaffected by this removal ## Related - Forgejo issue: #10 Closes #10

forgejo_admin added 1 commit

2026-03-14 13:00:06 +00:00

Remove dead listNoteSlugs() export from api.ts

ci/woodpecker/pr/woodpecker Pipeline was successful

Details

e46cf0515b

This function was superseded by the slug cache in src/lib/slugCache.ts
and has zero import references remaining.

Closes #10

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>

forgejo_admin commented

2026-03-14 13:00:22 +00:00

Author

Owner

Self-Review

Verdict: Clean -- no findings.

Diff is a pure 5-line deletion of the unused listNoteSlugs() function
No remaining imports or call sites (grep confirms zero references)
slugCache.ts calls listNotes directly, unaffected
npm run check and npm run build both pass
No functional or behavioral change to the application

## Self-Review **Verdict: Clean -- no findings.** - Diff is a pure 5-line deletion of the unused `listNoteSlugs()` function - No remaining imports or call sites (`grep` confirms zero references) - `slugCache.ts` calls `listNotes` directly, unaffected - `npm run check` and `npm run build` both pass - No functional or behavioral change to the application