Repository navigation
chore(docs): update AGENTS.md to reflect recent changes - #124
Conversation
PR #123 migrated staryl to @wolfstar/http-framework ^6.2.2, which moved the ApplyOptions decorator (and other utility decorators) out into a new @wolfstar/decorators package. Update the agent instructions so the middleware example and Key Dependencies list reflect the new import source. Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_013GQkdeyfeWfftSHscqFgnk
|
Note The title starts with
|
There was a problem hiding this comment.
ℹ️ No critical issues — one related doc left stale.
Reviewed changes
Docs-only review of the single commit; I checked each claim against package.json and src/middlewares/requestLogger.ts.
- Middleware example:
@ApplyOptionsnow imported from@wolfstar/decorators, matchingrequestLogger.ts:2. - Key Dependencies: new
@wolfstar/decoratorsentry, consistent with the^0.1.0dependency inpackage.json.
ℹ️ .github/CONTRIBUTING.md still documents the removed #utils/applyOptions shim
.github/CONTRIBUTING.md (around lines 216–224) still says middleware uses "the local @ApplyOptions decorator (#utils/applyOptions)" and imports from #utils/applyOptions. That module no longer exists in src/, so a contributor following it would write a broken import. This predates the PR, but it is the same drift this PR fixes in copilot-instructions.md, so it fits here.
Technical details
# Stale middleware docs in CONTRIBUTING.md
## Affected sites
- .github/CONTRIBUTING.md:216 — prose says "local `@ApplyOptions` decorator (`#utils/applyOptions`)"
- .github/CONTRIBUTING.md:221 — `import { ApplyOptions } from "#utils/applyOptions";`
## Required outcome
- Prose and example import match `@wolfstar/decorators`, as in `.github/copilot-instructions.md`.claude-sonnet-5-5 | 𝕏

Context
Weekly AGENTS.md maintenance review of PRs merged into
mainin the last 7 days (2026-10-04 – 2026-10-11).Only one PR merged in the window: #123 "chore(deps): update dependencies and migrate to http-framework 6". It bumped
@wolfstar/http-frameworkfrom^4.0.2to^6.2.2and, as part of that migration, moved theApplyOptionsutility decorator (and other utility decorators likeRequires*) out of@wolfstar/http-frameworkinto a new@wolfstar/decoratorspackage.src/middlewares/requestLogger.tsnow importsApplyOptionsfrom@wolfstar/decoratorsinstead of@wolfstar/http-framework..github/copilot-instructions.md(the fileAGENTS.mddelegates to) still documented the old state, which was now actively wrong and would mislead an agent writing new middleware.Changes
@ApplyOptionsdescription and code sample to import from@wolfstar/decoratorsinstead of@wolfstar/http-framework, and noted the package split happened in http-framework 6.@wolfstar/decorators, noting it was split out of@wolfstar/http-frameworkand that onlyApplyOptionsis currently used in this repo.No other recent changes (dependency version bumps, lockfile maintenance, workspace config) required documentation updates — development commands, naming conventions, directory structure, and other patterns in the file remain accurate.
🤖 Generated with Claude Code
https://claude.ai/code/session_013GQkdeyfeWfftSHscqFgnk
Generated by Claude Code


Confidence Score: 5/5
Safe to merge. The documentation matches the decorator import already used by the application and does not change runtime behavior, dependencies, or shared interfaces.Reviews (3) · Last reviewed commit: "Merge branch 'main' into claude/nice-dij..." · Reviewed by Greptile