feat: add NRNTN-TAI-Information protocol IE/extension ID

[donivtech]

Adds NGAP support consumed by omec-project/amf#684 (NR NTN access detection).

Changes

• ngapType/ProtocolIEID.go: add the protocol IE/extension identifier 287 assigned to NRNTN-TAI-Information per 3GPP TS 38.413. Follows the existing repo convention where ProtocolIEID* constants are the sole declaration for an NGAP identifier regardless of whether it is used as a ProtocolIE-ID, a ProtocolExtensionID, or both — NGAP's ASN.1 reuses one integer numbering space across both kinds (e.g. id-PSCellInformation = 149).
• extensions.go (new): add HasUserLocationInformationNRExtension(*ngapType.UserLocationInformationNR, id int64) bool, a generic helper that reports whether a protocol extension with the supplied identifier is present in UserLocationInformationNR.IEExtensions. Generic over the ID so any future protocol-extension presence check can compose with the ProtocolIEID* constants.
• extensions_test.go (new): 6 table-driven cases covering nil location, nil extensions, empty list, unrelated ID, matching ID, and matching ID among several.
• VERSION: drop -dev to cut release v1.9.1 so omec-project/amf#684 can pick these additions up via a go.mod bump (per maintainer request in the review thread).

Out of scope

The full Rel-17 IE type (NRNTNTAIInformation SEQUENCE) and the rest of the Rel-16/17/18 IE IDs are expected to land as part of the broader NGAP release uplift.

Test plan

• go build ./... clean
• go vet ./... clean
• go test ./... all pass
• golangci-lint run (v2.6.1) — 0 issues

[gab-arrobo]

@donivtech, please also remove "-dev" from VERSION file (as part of this PR) such that a new release is automatically created when this PR is merged. Thanks!

[donivtech]

Done. Added "Create patch release" commit dropping -dev from VERSION. Thanks!

[gab-arrobo]

Is there anything else from your AMF PR that should go in this NGAP PR?

[donivtech]

The constant was the only AMF ngap-side dependency. Though there's a small AMF-internal helper nrLocationHasNtnExt(*ngapType.UserLocationInformationNR) bool that traverses IEExtensions for the ID, but it's 8 lines, called from a single site, that could be moved to ngap. But there are no similar helper functions in ngap. Otherwise nothing else.

[gab-arrobo]

The constant was the only AMF ngap-side dependency. Though there's a small AMF-internal helper nrLocationHasNtnExt(*ngapType.UserLocationInformationNR) bool that traverses IEExtensions for the ID, but it's 8 lines, called from a single site, that could be moved to ngap. But there are no similar helper functions in ngap. Otherwise nothing else.

Would it make sense to move it here? I am asking this because it would be good to keep NGAP related stuff in it own repo such that when time comes to upgrade N2 release, we know it is there and not "hidden" in some other repo.. what so you think?

[donivtech]

Yes sound good. Will move it as HasUserLocationInformationNRExtension(*ngapType.UserLocationInformationNR, id int64) bool (top-level ngap package, alongside the existing Decoder/Encoder). Generic over the ID instead of ntn-specific so the helper is reusable for any future protocol-extension presence check. Will push shortly

[Copilot]

Pull request overview

Adds support for the NRNTN-TAI-Information identifier so downstream components (e.g., AMF) can reference a canonical NGAP constant when handling NR NTN-related location extensions.

Changes:

• Add ProtocolIEIDNRNTNTAIInformation constant with value 287.
• Introduce HasUserLocationInformationNRExtension helper for checking presence of an NR UserLocationInformation extension ID.
• Add unit tests for the new helper and update VERSION from 1.9.1-dev to 1.9.1.

Reviewed changes

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

File	Description
ngapType/ProtocolIEID.go	Adds the new NRNTN-TAI-Information identifier constant (287).
extensions.go	Adds an exported helper to detect a specific extension ID in UserLocationInformationNR.IEExtensions.
extensions_test.go	Adds coverage for the new helper across nil/empty/matching/non-matching cases.
VERSION	Switches from a -dev version to a release version, which can trigger release automation.

---

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

[donivtech]

Addressing review comments:
1 , 5 (PR scope and description): The description is stale. Will update with correct scope.
3 (doc comment): Will tighten to refer to "ID" / "ProtocolExtensionID" and explicitly note the shared NGAP IE/extension numbering space.
2, 4 (naming): The repo's existing convention follows shared numbering space between ProtocolIE-ID and ProtocolExtensionID e.g id-PSCellInformation = 149 is the same number used in both contexts. Adding a ProtocolExtensionIDNRNTNTAIInformation alias just for our one constant would be inconsistent with the rest.

[gab-arrobo]

+1