refactor: rename fetchWrapper to middleware and move to client directory

- Renamed fetchWrapper.ts to middleware.ts for cleaner naming
- Renamed FetchMiddleware type to Middleware (kept deprecated aliases)
- Moved middleware files from src/shared/ to src/client/ since this is client-specific
- Updated all imports to reflect new location
- Fixed test for short-circuit responses

The middleware is now properly located in the client directory where it belongs,
separate from any server-side middleware.

🤖 Generated with [Claude Code](https://claude.ai/code)

Co-Authored-By: Claude <[email protected]>

Author: dsp-ant

src/shared/fetchWrapper.test.ts renamed to src/client/middleware.test.ts

@@ -3,9 +3,9 @@ import {
   withLogging,
   applyMiddleware,
   createMiddleware,
-} from "./fetchWrapper.js";
-import { OAuthClientProvider } from "../client/auth.js";
-import { FetchLike } from "./transport.js";
+} from "./middleware.js";
+import { OAuthClientProvider } from "./auth.js";
+import { FetchLike } from "../shared/transport.js";
 
 jest.mock("../client/auth.js", () => {
   const actual = jest.requireActual("../client/auth.js");
@@ -16,7 +16,7 @@ jest.mock("../client/auth.js", () => {
   };
 });
 
-import { auth, extractResourceMetadataUrl } from "../client/auth.js";
+import { auth, extractResourceMetadataUrl } from "./auth.js";
 
 const mockAuth = auth as jest.MockedFunction<typeof auth>;
 const mockExtractResourceMetadataUrl =
@@ -1091,6 +1091,8 @@ describe("createMiddleware", () => {
 
     // Test normal route
     mockFetch.mockResolvedValue(new Response("fresh data", { status: 200 }));
+    const normalResponse = await enhancedFetch("https://example.com/normal/data");
+    expect(await normalResponse.text()).toBe("fresh data");
     expect(mockFetch).toHaveBeenCalledTimes(1);
   });
 

src/shared/fetchWrapper.ts renamed to src/client/middleware.ts

@@ -3,19 +3,24 @@ import {
   extractResourceMetadataUrl,
   OAuthClientProvider,
   UnauthorizedError,
-} from "../client/auth.js";
-import { FetchLike } from "./transport.js";
+} from "./auth.js";
+import { FetchLike } from "../shared/transport.js";
 
 /**
  * Middleware function that wraps and enhances fetch functionality.
  * Takes a fetch handler and returns an enhanced fetch handler.
  */
-export type FetchMiddleware = (next: FetchLike) => FetchLike;
+export type Middleware = (next: FetchLike) => FetchLike;
 
 /**
- * @deprecated Use FetchMiddleware instead
+ * @deprecated Use Middleware instead
  */
-export type FetchWrapper = FetchMiddleware;
+export type FetchWrapper = Middleware;
+
+/**
+ * @deprecated Use Middleware instead
+ */
+export type FetchMiddleware = Middleware;
 
 /**
  * Creates a fetch wrapper that handles OAuth authentication automatically.
@@ -44,7 +49,7 @@ export type FetchWrapper = FetchMiddleware;
  * @returns A fetch middleware function
  */
 export const withOAuth =
-  (provider: OAuthClientProvider, baseUrl?: string | URL): FetchMiddleware =>
+  (provider: OAuthClientProvider, baseUrl?: string | URL): Middleware =>
   (next) => {
     return async (input, init) => {
       const makeRequest = async (): Promise<Response> => {
@@ -167,7 +172,7 @@ export type LoggingOptions = {
  * @param options - Logging configuration options
  * @returns A fetch middleware function
  */
-export const withLogging = (options: LoggingOptions = {}): FetchMiddleware => {
+export const withLogging = (options: LoggingOptions = {}): Middleware => {
   const {
     logger,
     includeRequestHeaders = false,
@@ -282,8 +287,8 @@ export const withLogging = (options: LoggingOptions = {}): FetchMiddleware => {
  * @returns A single composed middleware function
  */
 export const applyMiddleware = (
-  ...middleware: FetchMiddleware[]
-): FetchMiddleware => {
+  ...middleware: Middleware[]
+): Middleware => {
   return (next) => {
     return middleware.reduce((handler, mw) => mw(handler), next);
   };
@@ -353,6 +358,6 @@ export const createMiddleware = (
     input: string | URL,
     init?: RequestInit,
   ) => Promise<Response>,
-): FetchMiddleware => {
+): Middleware => {
   return (next) => (input, init) => handler(next, input as string | URL, init);
 };