feat(http-request): rename helpers and add default headers

- Rename httpGetJson → httpJson (supports all HTTP methods)
- Rename httpGetText → httpText (supports all HTTP methods)
- Add default headers for httpJson:
  - Accept: application/json (always)
  - Content-Type: application/json (when body present)
- Add default headers for httpText:
  - Accept: text/plain (always)
  - Content-Type: text/plain (when body present)
- User-provided headers override defaults
- Add 8 new tests for default header behavior (67 total tests)

BREAKING CHANGE: httpGetJson and httpGetText have been removed. Use httpJson and httpText instead.

Author: jdalton

src/http-request.ts

@@ -1034,8 +1034,12 @@ async function httpDownloadAttempt(
 }
 
 /**
- * Perform a GET request and parse JSON response.
- * Convenience wrapper around `httpRequest` for common JSON API calls.
+ * Perform an HTTP request and parse JSON response.
+ * Convenience wrapper around `httpRequest` for JSON API calls.
+ * Automatically sets appropriate headers for JSON requests:
+ * - `Accept: application/json` (always)
+ * - `Content-Type: application/json` (when body is present)
+ * User-provided headers override these defaults.
  *
  * @template T - Expected JSON response type (defaults to `unknown`)
  * @param url - The URL to request (must start with http:// or https://)
@@ -1045,35 +1049,65 @@ async function httpDownloadAttempt(
  *
  * @example
  * ```ts
- * // Simple JSON GET
- * const data = await httpGetJson('https://api.example.com/data')
+ * // Simple JSON GET (automatically sets Accept: application/json)
+ * const data = await httpJson('https://api.example.com/data')
  * console.log(data)
  *
  * // With type safety
  * interface User { id: number; name: string; email: string }
- * const user = await httpGetJson<User>('https://api.example.com/user/123')
+ * const user = await httpJson<User>('https://api.example.com/user/123')
  * console.log(user.name, user.email)
  *
- * // With custom headers
- * const data = await httpGetJson('https://api.example.com/data', {
- *   headers: {
- *     'Authorization': 'Bearer token123',
- *     'Accept': 'application/json'
- *   }
+ * // POST with JSON body (automatically sets Content-Type: application/json)
+ * const result = await httpJson('https://api.example.com/users', {
+ *   method: 'POST',
+ *   body: JSON.stringify({ name: 'Alice', email: 'alice@example.com' })
  * })
  *
- * // With retries
- * const data = await httpGetJson('https://api.example.com/data', {
+ * // With custom headers and retries
+ * const data = await httpJson('https://api.example.com/data', {
+ *   headers: {
+ *     'Authorization': 'Bearer token123'
+ *   },
  *   retries: 3,
  *   retryDelay: 1000
  * })
  * ```
  */
-export async function httpGetJson<T = unknown>(
+export async function httpJson<T = unknown>(
   url: string,
   options?: HttpRequestOptions | undefined,
 ): Promise<T> {
-  const response = await httpRequest(url, { ...options, method: 'GET' })
+  const {
+    body,
+    headers = {},
+    ...restOptions
+  } = {
+    __proto__: null,
+    ...options,
+  } as HttpRequestOptions
+
+  // Set default headers for JSON requests
+  const defaultHeaders: Record<string, string> = {
+    Accept: 'application/json',
+  }
+
+  // Add Content-Type when body is present
+  if (body !== undefined) {
+    defaultHeaders['Content-Type'] = 'application/json'
+  }
+
+  // Merge headers: user headers override defaults
+  const mergedHeaders = {
+    ...defaultHeaders,
+    ...headers,
+  }
+
+  const response = await httpRequest(url, {
+    body,
+    headers: mergedHeaders,
+    ...restOptions,
+  })
 
   if (!response.ok) {
     throw new Error(`HTTP ${response.status}: ${response.statusText}`)
@@ -1087,8 +1121,12 @@ export async function httpGetJson<T = unknown>(
 }
 
 /**
- * Perform a GET request and return text response.
+ * Perform an HTTP request and return text response.
  * Convenience wrapper around `httpRequest` for fetching text content.
+ * Automatically sets appropriate headers for text requests:
+ * - `Accept: text/plain` (always)
+ * - `Content-Type: text/plain` (when body is present)
+ * User-provided headers override these defaults.
  *
  * @param url - The URL to request (must start with http:// or https://)
  * @param options - Request configuration options
@@ -1097,32 +1135,68 @@ export async function httpGetJson<T = unknown>(
  *
  * @example
  * ```ts
- * // Fetch HTML
- * const html = await httpGetText('https://example.com')
+ * // Fetch HTML (automatically sets Accept: text/plain)
+ * const html = await httpText('https://example.com')
  * console.log(html.includes('<!DOCTYPE html>'))
  *
  * // Fetch plain text
- * const text = await httpGetText('https://example.com/file.txt')
+ * const text = await httpText('https://example.com/file.txt')
  * console.log(text)
  *
- * // With custom headers
- * const text = await httpGetText('https://example.com/data.txt', {
+ * // POST with text body (automatically sets Content-Type: text/plain)
+ * const result = await httpText('https://example.com/api', {
+ *   method: 'POST',
+ *   body: 'raw text data'
+ * })
+ *
+ * // With custom headers (override defaults)
+ * const text = await httpText('https://example.com/data.txt', {
  *   headers: {
- *     'Authorization': 'Bearer token123'
+ *     'Authorization': 'Bearer token123',
+ *     'Accept': 'text/html'  // Override default Accept header
  *   }
  * })
  *
  * // With timeout
- * const text = await httpGetText('https://example.com/large-file.txt', {
+ * const text = await httpText('https://example.com/large-file.txt', {
  *   timeout: 60000 // 1 minute
  * })
  * ```
  */
-export async function httpGetText(
+export async function httpText(
   url: string,
   options?: HttpRequestOptions | undefined,
 ): Promise<string> {
-  const response = await httpRequest(url, { ...options, method: 'GET' })
+  const {
+    body,
+    headers = {},
+    ...restOptions
+  } = {
+    __proto__: null,
+    ...options,
+  } as HttpRequestOptions
+
+  // Set default headers for text requests
+  const defaultHeaders: Record<string, string> = {
+    Accept: 'text/plain',
+  }
+
+  // Add Content-Type when body is present
+  if (body !== undefined) {
+    defaultHeaders['Content-Type'] = 'text/plain'
+  }
+
+  // Merge headers: user headers override defaults
+  const mergedHeaders = {
+    ...defaultHeaders,
+    ...headers,
+  }
+
+  const response = await httpRequest(url, {
+    body,
+    headers: mergedHeaders,
+    ...restOptions,
+  })
 
   if (!response.ok) {
     throw new Error(`HTTP ${response.status}: ${response.statusText}`)