MINIFICPP-2906 Add guid nodeid support for OPC processors and some additional fixes

[lordgamez]

• GUID nodeid support for all OPC processors
• Make Parent node ID property optional for PutOPCProcessor
• Move node id parsing to onSchedule where possible
• Add missing tests for uncovered use cases like multiple node id types
• Add namespace index attribute to output flow files in FetchOPCProcessor
• Additional refactoring of common code

https://issues.apache.org/jira/browse/MINIFICPP-2906

Depends on #2265

---

Thank you for submitting a contribution to Apache NiFi - MiNiFi C++.

In order to streamline the review of the contribution we ask you to ensure the following steps have been taken:

For all changes:

• Is there a JIRA ticket associated with this PR? Is it referenced in the commit message?

• Does your PR title start with MINIFICPP-XXXX where XXXX is the JIRA number you are trying to resolve? Pay particular attention to the hyphen "-" character.

• Has your PR been rebased against the latest commit within the target branch (typically main)?

• Is your initial contribution a single, squashed commit?

For code changes:

• If adding new dependencies to the code, are these dependencies licensed in a way that is compatible for inclusion under ASF 2.0?
• If applicable, have you updated the LICENSE file?
• If applicable, have you updated the NOTICE file?

For documentation related changes:

• Have you ensured that format looks appropriate for the output in which it is rendered?

Note:

Please ensure that once the PR is submitted, you check GitHub Actions CI results for build issues and submit an update to your PR as soon as possible.

[Copilot]

🟡 Changes recommended

Numeric NodeIds are parsed incorrectly and cached parent-path resolutions can become stale after server changes.

Get a fresh assessment by requesting another Copilot review.

Pull request overview

Adds GUID NodeId support across OPC processors and makes PutOPC parent configuration optional.

Changes:

• Centralizes NodeId parsing and GUID handling.
• Supports updating nodes without a configured parent.
• Expands OPC tests, documentation, and manifest metadata.

File summaries

File	Description
PROCESSORS.md	Documents GUID and optional parent properties.
extensions/opc/tests/PutOPCProcessorTests.cpp	Tests GUID and parentless updates.
extensions/opc/tests/OpcUaTestServer.h	Adds GUID and ambiguous test nodes.
extensions/opc/tests/FetchOPCProcessorTests.cpp	Tests multiple NodeId types and traversal cases.
extensions/opc/src/PutOPCProcessor.cpp	Refactors parent and target handling.
extensions/opc/src/OPCCommon.cpp	Adds common NodeId construction and GUID attributes.
extensions/opc/src/FetchOPCProcessor.cpp	Uses common scheduled parsing.
extensions/opc/src/FetchOPCHistory.cpp	Reuses base NodeId parsing.
extensions/opc/src/BaseOPCProcessor.cpp	Centralizes processor NodeId parsing.
extensions/opc/include/PutOPCProcessor.h	Makes parent properties optional.
extensions/opc/include/OPCCommon.h	Declares common NodeId builder.
extensions/opc/include/FetchOPCProcessor.h	Enables GUID NodeIds.
extensions/opc/include/FetchOPCHistory.h	Removes duplicate state.
extensions/opc/include/BaseOPCProcessor.h	Adds shared parsed-node state.
.github/references/ubuntu_22_04_clang_arm_manifest.json	Updates generated processor metadata.

Review details

• Files reviewed: 15/15 changed files
• Comments generated: 2
• Review effort level: Balanced

---

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.