Skip to content

chore: quality & housekeeping sweep (TypeScript / Next.js) - #583

Merged
fo0 merged 2 commits into
mainfrom
claude/cool-babbage-joarzp
Sep 6, 2026
Merged

fo0 merged 2 commits into
mainfrom
claude/cool-babbage-joarzp

Conversation

@claude

@claude claude Bot commented Sep 6, 2026

Copy link
Copy Markdown
Contributor

Autonomous quality & housekeeping sweep over the TypeScript / Next.js stack. Two findings raised, both fixed and both mechanical; the full check chain is green on the final commit.

Merged findings

# Pass Datei(en) Befund Fix Aufwand Empfehlung
1 E (Docs) agent_docs/project-structure.md 7 modules exist under src/ but are absent from the structure tree, whose own header asks to be updated on every add/move/remove: components/ErrorBoundary.tsx, components/SidebarResizer.tsx, utils/access-log-filter.ts, utils/download.ts, utils/editor-draft.ts, utils/file-import.ts, utils/sidebar-width.ts 7 tree entries added at their alphabetical position in the utils/ list (the components/ list is grouped, not sorted, so the resizer sits next to Sidebar.tsx and the crash guard leads the block); each description is taken from the module's own header comment S auto-fix
2 Security .gitignore Only the bare .env was ignored. Next.js also loads .env.local, .env.production and .env.*.local, so any of those would have been offered by git add with nothing stopping the commit. .dockerignore already excludes .env.* — the two ignore files were out of step .env.* added with a !.env.example negation. Verified with git check-ignore: .env.example is still not ignored, .env.local and .env.production.local now are. No tracked file changes state S auto-fix

No source file under src/ is touched by this PR.

Tooling-Status

Tool Aktiv Konfiguriert in Lief grün
Prettier 3.9.6 (format:check) ja .prettierrc.json, .prettierignore ja — "All matched files use Prettier code style!"
TypeScript (tsc --noEmit) ja tsconfig.json ja — exit 0
ESLint 9 flat (lint) ja eslint.config.js ja — exit 0, 0 errors / 27 pre-existing a11y warnings
vitest 4 (test) ja vitest.config.ts ja — 72 files, 637 passed, 5 skipped
Next.js build (build) ja next.config.ts ja — exit 0

Chain run in this repo's own order (CLAUDE.md → Commands, mirroring docker-publish.yml): npm ciformat:checktsc --noEmitlinttestbuild.

Deferred

Both are recorded in #582.

Keine Version-Bumps enthalten

package.json and package-lock.json are untouched — git diff -- package.json package-lock.json is empty. Dependency version bumps and Dependabot PRs (#578, #476, #437, #360, #125) are out of scope for this routine and were not touched.

Closes #582

🤖 Generated with Claude Code

https://claude.ai/code/session_016oZDrMULeCJTxz762nC8aH


Generated by Claude Code

`agent_docs/project-structure.md` asks to be updated whenever a file is
added, moved or removed, but seven modules had landed without an entry:
`components/ErrorBoundary.tsx`, `components/SidebarResizer.tsx` and the
utils `access-log-filter.ts`, `download.ts`, `editor-draft.ts`,
`file-import.ts`, `sidebar-width.ts`.

Each is inserted at its alphabetical position in the `utils/` list (the
`components/` list is grouped, not sorted, so the resizer sits next to
`Sidebar.tsx` and the crash guard leads the block) with a one-line
description taken from the module's own header comment. Documentation
only -- no source file is touched.

Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_016oZDrMULeCJTxz762nC8aH
`.gitignore` listed the bare `.env` only. Next.js also loads `.env.local`,
`.env.production` and `.env.*.local`, so a developer following the framework's
own convention would have had that file offered by `git add` with nothing
stopping the commit. `.dockerignore` already excludes `.env.*`; the two ignore
files were out of step.

Adds `.env.*` with a `!.env.example` negation so the committed template stays
tracked. Verified with `git check-ignore`: `.env.example` is still not ignored,
`.env.local` and `.env.production.local` now are. No tracked file changes state.

Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_016oZDrMULeCJTxz762nC8aH
@fo0
fo0 merged commit 54a3f6f into main Sep 6, 2026
3 checks passed
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.

chore: quality & housekeeping sweep (TypeScript / Next.js)

2 participants