Repository navigation
chore(docs): update AGENTS.md to reflect recent changes - #124
RedStar071 wants to merge 1 commit into
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 corrected guidance matches the working middleware and shared package exports, and the updated example passes compilation and runtime checks.T-Rex evidence
Command output from the check
Command output from the check
Command output from the check
Command output from the check
Command output from the check
Command output from the check
Evidence from the check
Evidence from the check
Evidence from the check
Evidence from the check
Evidence from the check
Evidence from the check
Evidence from the check
Evidence from the check
Evidence from the check
Evidence from the check
Evidence from the check
Reviews (1) · Last reviewed commit: "docs: update AGENTS.md guidance for http..." · Reviewed by Greptile