NIFI-15815 connector details page

[scottyaslan]

Summary

NIFI-15815

Tracking

Please complete the following tracking steps prior to pull request creation.

Issue Tracking

• Apache NiFi Jira issue created

Pull Request Tracking

• Pull Request title starts with Apache NiFi Jira issue number, such as NIFI-00000
• Pull Request commit message starts with Apache NiFi Jira issue number, as such NIFI-00000
• Pull request contains commits signed with a registered key indicating Verified status

Pull Request Formatting

• Pull Request based on current revision of the main branch
• Pull Request refers to a feature branch with one commit containing changes

Verification

Please indicate the verification steps performed prior to pull request creation.

Build

• Build completed using ./mvnw clean install -P contrib-check
  • JDK 21
  • JDK 25

Licensing

• New dependencies are compatible with the Apache License 2.0 according to the License Policy
• New dependencies are documented in applicable LICENSE and NOTICE files

Documentation

• Documentation formatting appears as expected in rendered files

[mcgilman]

Reviewing...

[mcgilman]

Nice, clean addition — the split into ConnectorDetail + ConnectorDetailsContent is the right shape, sandbox flags are correctly tightened for a read-only view, and the spec coverage on the visibility branches is thorough.

A few comments below; the only one with real substance is the getFrameSource sanitize-then-bypass pattern, which is duplicated from ConnectorConfigure and worth fixing in both. The rest are small consistency/optional nits.

[mcgilman]

Minor consistency: ConnectorConfigure uses <p class="text-base" data-qa="error-message">{{ errorMessage }}</p> for the same role. Without the <p> + text-base, the error text picks up the surrounding default typography instead of Material body sizing. Suggest:

<p class="text-base" data-qa="error-message">{{ errorMessage }}</p>

[scottyaslan]

This is exactly equivalent typogaphically. We do not typically use <p class="text-base"></p> elsewhere in the codebase.

[mcgilman]

It's used in these places:

1 - https://github.com/apache/nifi/blob/main/nifi-frontend/src/main/frontend/apps/nifi/src/app/pages/connectors/ui/connector-configure/connector-configure.component.html#L34
2 - https://github.com/apache/nifi/blob/main/nifi-frontend/src/main/frontend/libs/shared/src/components/connector-wizard/connector-wizard.component.html#L32
3 - https://github.com/apache/nifi/blob/main/nifi-frontend/src/main/frontend/libs/shared/src/components/connector-wizard/connector-wizard.component.html#L115
4 - https://github.com/apache/nifi/blob/main/nifi-frontend/src/main/frontend/libs/shared/src/components/connector-wizard/connector-configuration-step/connector-configuration-step.component.html#L55