Skip to content

[Patch] Added Support of requiresStreamAddress and requiresStreamParameterAddress#977

Open
rajatavams wants to merge 1 commit intoOfficeDev:masterfrom
rajatavams:user/rajatavams/support-requiresStreamAddress-requiresStreamParameterAddresses
Open

[Patch] Added Support of requiresStreamAddress and requiresStreamParameterAddress#977
rajatavams wants to merge 1 commit intoOfficeDev:masterfrom
rajatavams:user/rajatavams/support-requiresStreamAddress-requiresStreamParameterAddresses

Conversation

@rajatavams
Copy link

@rajatavams rajatavams commented Feb 23, 2026

Adds explicit support for @requiresStreamAddress and @requiresStreamParameterAddresses tags in custom function metadata parsing.

Context GitHub issue #6499

This update:

  1. recognizes both tags directly
  2. maps them to metadata options (requiresStreamAddress, requiresStreamParameterAddresses),
  3. validates that these stream-specific tags are used only on streaming functions,
  4. updates conflict validation/error messaging paths to include the new tags,
  5. adds test coverage for valid explicit stream tags and invalid non-stream usage.
  6. this does not impact existing behaviour as mentioned in the context section (@ requiresAddress / @ requiresParamterAddress + @ Streaming = requiresStreamAddress / requiresStreamParamterAddress) to ensure backward compatibility.

Do these changes impact command syntax of any of the packages?
No.

Do these changes impact documentation?
Need to add @ requiresStreamAddressand @ requiresStreamParameterAddresses in Supported JSDoc Tags List.

Validation/testing performed:

Ran package tests in custom-functions-metadata with npm test.
Result: 137 passing.

Manual Testing Done Via custom-functions-metadata generate command.

@rajatavams
Copy link
Author

@rajatavams please read the following Contributor License Agreement(CLA). If you agree with the CLA, please reply with the following information.

@microsoft-github-policy-service agree [company="{your company}"]

Options:

  • (default - no company specified) I have sole ownership of intellectual property rights to my Submissions and I am not making Submissions in the course of work for my employer.
@microsoft-github-policy-service agree
  • (when company given) I am making Submissions in the course of work for my employer (or my employer has intellectual property rights in my Submissions by contract or applicable law). I have permission from my employer to make Submissions and enter into this Agreement on behalf of my employer. By signing below, the defined term “You” includes me and my employer.
@microsoft-github-policy-service agree company="Microsoft"

Contributor License Agreement

@microsoft-github-policy-service agree company="Microsoft"

@rajatavams rajatavams marked this pull request as ready for review February 24, 2026 04:41
@rajatavams rajatavams requested a review from a team as a code owner February 24, 2026 04:41
@rajatavams rajatavams changed the title Added Support of requiresStreamAddress and requiresStreamParameterAddress [Patch] Added Support of requiresStreamAddress and requiresStreamParameterAddress Feb 24, 2026
@millerds
Copy link
Collaborator

/azurepipelines run

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

Copy link
Collaborator

@millerds millerds left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks like the package-lock file is not correct.

@rajatavams rajatavams marked this pull request as draft February 27, 2026 07:45
@rajatavams rajatavams force-pushed the user/rajatavams/support-requiresStreamAddress-requiresStreamParameterAddresses branch from ad428dd to d2becdd Compare February 27, 2026 13:13
@rajatavams
Copy link
Author

/azurepipelines run

@azure-pipelines
Copy link

Commenter does not have sufficient privileges for PR 977 in repo OfficeDev/Office-Addin-Scripts

@rajatavams rajatavams marked this pull request as ready for review February 27, 2026 13:17
@rajatavams
Copy link
Author

/azurepipelines run

@azure-pipelines
Copy link

Commenter does not have sufficient privileges for PR 977 in repo OfficeDev/Office-Addin-Scripts

@rajatavams rajatavams force-pushed the user/rajatavams/support-requiresStreamAddress-requiresStreamParameterAddresses branch from d2becdd to d7d14b9 Compare February 27, 2026 15:50
@millerds
Copy link
Collaborator

/azurepipelines run

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@millerds
Copy link
Collaborator

@rajatavams you have to be part of the DevX group in order to trigger the automation (which would also trigger automatically if you were part of the group).

Looks like the lock file still needs correct updating.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants