Add custom User-Agent header to Octagon API requests

[melvinmt]

Summary by CodeRabbit

• Chores
  • Updated API request headers to include client identification information for improved service monitoring and tracking.

_{✏️ Tip: You can customize this high-level summary in your review settings.}

[coderabbitai]

Walkthrough

Added HTTP header "User-Agent": "Octagon-Excel-AddIn/1.2.0" to authenticated API requests in the OctagonApiService. The header is included alongside existing "Content-Type" and Authorization headers when making requests to the /v1/responses endpoint. No changes to API key handling, request body structure, error handling, or control flow.

Changes

Cohort / File(s)	Summary
User-Agent Header Addition src/api/octagonApi.ts	Added HTTP header "User-Agent": "Octagon-Excel-AddIn/1.2.0" to authenticated API requests to /v1/responses endpoint, included alongside existing "Content-Type" and Authorization headers

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~3 minutes

Pre-merge checks and finishing touches

✅ Passed checks (3 passed)

Check name	Status	Explanation
Description Check	✅ Passed	Check skipped - CodeRabbit’s high-level summary is enabled.
Title check	✅ Passed	The pull request title accurately and clearly summarizes the main change: adding a custom User-Agent header to API requests.
Docstring Coverage	✅ Passed	No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

✨ Finishing touches

• 📝 Generate docstrings

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Post copyable unit tests in a comment
• Commit unit tests in branch user-agent

---

_{Comment @coderabbitai help to get the list of available commands and usage tips.}

[coderabbitai]

Actionable comments posted: 0

🧹 Nitpick comments (1)

src/api/octagonApi.ts (1)

159-159: Extract the hardcoded version to prevent version drift.

The version "1.2.0" is hardcoded in the User-Agent string at line 159. This creates a maintenance burden—every time the add-in version is updated in package.json, this line must be manually changed, risking version mismatches.

Consider extracting it as a constant at the file level or reading it dynamically from package.json.

Proposed refactor: Extract version as constant

Define a constant at the top of the file after imports:

 import Logger from "../utils/logger";
 import { getTextFormat, parseTextFormat } from "./format";
 import { AgentRequest, AgentResponse, AgentType, OutputFormat } from "./types";
+
+// Version for User-Agent header (should match package.json version)
+const ADDIN_VERSION = "1.2.0";

Then update line 159 to use the constant:

     const headers = new Headers({
       "Content-Type": "application/json",
       Authorization: `Bearer ${token}`,
-      "User-Agent": "Octagon-Excel-AddIn/1.2.0",
+      "User-Agent": `Octagon-Excel-AddIn/${ADDIN_VERSION}`,
     });

📜 Review details

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between 8205d5d and 6f8bc30.

📒 Files selected for processing (1)

• src/api/octagonApi.ts