chore: enable noUncheckedIndexedAccess

[blva]

closes #184

[Copilot]

Pull Request Overview

This PR enables TypeScript's noUncheckedIndexedAccess by adding optional chaining and default fallbacks across tests and source files to avoid undefined index accesses.

• Added ?. or ?? guards for array indexing in tests and parsing logic.
• Introduced null checks and default values in application code (tools, atlas helpers, scripts).
• Updated logging and error handling for missing tool registrations and Atlas project creation.

Reviewed Changes

Copilot reviewed 30 out of 30 changed files in this pull request and generated 4 comments.

Show a summary per file

File	Description
tests/integration/tools/mongodb/.../*.test.ts	Added ?. guards to array index accesses in assertions
tests/integration/tools/mongodb/metadata/dbStats.test.ts	Defaulted JSON.parse input with ?? "{}"
tests/integration/tools/atlas/atlasHelpers.ts	Guarded header indexing in parseTable
src/tools/tool.ts	Logged and early-returned on missing existingTool
src/tools/atlas/read/listProjects.ts	Filtered out empty org IDs and defaulted orgName
src/tools/atlas/create/createProject.ts	Checked firstOrg.id before assignment
src/common/atlas/cluster.ts	Refactored instanceSize fallback logic
scripts/apply.ts	Cast OpenAPI refs and null-checked response content

Comments suppressed due to low confidence (2)

tests/integration/tools/mongodb/metadata/explain.test.ts:92

• Since you've already asserted that content has length ≥ 2, use the non-null assertion (content[0]!.text) instead of optional chaining to avoid masking indexing errors.

expect(content[0]?.text).toEqual(

tests/integration/tools/mongodb/metadata/dbStats.test.ts:33

• Prefer using a non-null assertion (elements[1]!.text) here since elements is asserted to have length 2; this makes test failures more explicit if the element is missing.

const stats = JSON.parse(elements[1]?.text ?? "{}") as {

[Copilot]

The headers array is always defined here; you can remove the runtime if check and use row[headers[index]!] = cell; to simplify the code.

Suggested change

	if (headers) {
	row[headers[index] ?? ""] = cell;
	}
	row[headers[index]!] = cell;

[blva]

'headers' is possibly 'undefined'.ts(18048)