feat(integration-service): add Connectors service

[maninder-uipath]

Summary

PR 2/4 of the Integration Service stack. Adds the Connectors service.

• New service via @uipath/uipath-typescript/is-connectors: getAll, getById, getDefaultConnection, getConnections
• CONNECTOR_ENDPOINTS constants; delegates connection lookups to the Connections service
• Unit tests, createMockConnector fixture, OAuth scope docs, mkdocs nav

Stack

• Base: feat/is-connections (PR feat(integration-service): add Connections service #554) — merge that first; GitHub will retarget this to main.
• Depends on Connections (connectors.ts imports ConnectionsService).

🤖 Generated with Claude Code

[claude]

Single-type alias (type X = Y) breaks TypeDoc rendering per the project convention:

"The only place interface extends is required is single-type aliases (type X = Y), which break TypeDoc — use export interface EntityUpdateRecordResponse extends EntityRecord {} instead."

Suggested change

	export type ConnectorGetResponse = RawConnectorGetResponse;
	export interface ConnectorGetResponse extends RawConnectorGetResponse {}

[claude]

Convention: "Use enums for fixed value sets — NEVER leave raw strings/numbers."

The JSDoc lists GA, BETA, CUSTOM — these look like a closed API enum, not open-ended strings. If the API documents a fixed set, declare a ConnectorLifecycleStage enum and type this field with it.

If the API can return arbitrary strings (e.g. vendor-specific stages on custom connectors), add a comment explaining that and the string type is fine.

[claude]

Missing error propagation test for getDefaultConnection. Convention: "Test both success and error scenarios for every public method."

getAll and getById both have a 'should propagate API errors' test — getDefaultConnection needs one too:

it('should propagate API errors', async () => {
  mockApiClient.get.mockRejectedValue(createMockError(IS_TEST_CONSTANTS.ERROR_CONNECTION_NOT_FOUND));
  await expect(
    service.getDefaultConnection(IS_TEST_CONSTANTS.CONNECTOR_KEY),
  ).rejects.toThrow(IS_TEST_CONSTANTS.ERROR_CONNECTION_NOT_FOUND);
});

[claude]

Same gap: getConnections is missing an error propagation test. Convention: "Test both success and error scenarios for every public method."

it('should propagate API errors', async () => {
  mockApiClient.get.mockRejectedValue(createMockError(TEST_CONSTANTS.ERROR_MESSAGE));
  await expect(
    service.getConnections(IS_TEST_CONSTANTS.CONNECTOR_KEY),
  ).rejects.toThrow(TEST_CONSTANTS.ERROR_MESSAGE);
});

[claude]

Review findings

New inline comments posted this run:

1. connectors.models.ts line 19 — single-type alias breaks TypeDoc; use interface extends.
2. connectors.types.ts line 48 — lifeCycleStage should be an enum if GA/BETA/CUSTOM is a closed set.
3. connectors.test.ts line 144 — getDefaultConnection is missing an error-propagation test.
4. connectors.test.ts line 182 — getConnections is missing an error-propagation test.

Missing file:

5. No integration test file — convention requires tests/integration/shared/integration-service/connectors.test.ts covering getAll, getById, getDefaultConnection, and getConnections.