Make raiseIntent context argument optional - #1983
Open
Roaders wants to merge 1 commit into
Open
Conversation
❌ Deploy Preview for fdc3 failed. Why did it fail? →
|
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## main #1983 +/- ##
=======================================
Coverage 95.48% 95.48%
=======================================
Files 85 85
Lines 6705 6711 +6
Branches 818 897 +79
=======================================
+ Hits 6402 6408 +6
Misses 303 303 ☔ View full report in Codecov by Harness. 🚀 New features to boost your workflow:
|
The context argument of raiseIntent is now optional (context?: Context | null). Apps that only raise an intent to obtain a result no longer need to pass the fdc3.nothing context explicitly. When context is omitted (or null/undefined is passed) the Desktop Agent proxy substitutes the fdc3.nothing context type, so the wire protocol is unchanged, the IntentHandler is untouched, and intent listeners always receive a concrete context. This is an additive, non-breaking change for TypeScript applications. - fdc3-standard: make context optional/null on the DesktopAgent interface and update the JSDoc and examples - fdc3-agent-proxy: substitute fdc3.nothing when no context is provided - website docs: mirror the interface change - tests: add proxy scenarios for the no-context and null-context cases - CHANGELOG: add entry under [Unreleased] > Changed
Roaders
force-pushed
the
raiseIntent-context-optional
branch
from
July 23, 2026 07:45
277eea1 to
07aaa53
Compare
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
The context argument of raiseIntent is now optional (context?: Context | null). Apps that only raise an intent to obtain a result no longer need to pass the fdc3.nothing context explicitly.
When context is omitted (or null/undefined is passed) the Desktop Agent proxy substitutes the fdc3.nothing context type, so the wire protocol is unchanged, the IntentHandler is untouched, and intent listeners always receive a concrete context. This is an additive, non-breaking change for TypeScript applications.
Describe your change
Related Issue
resolves #1708
Contributor License Agreement
Review Checklist
DesktopAgent,Channel,PrivateChannel,Listener,Bridging)?JSDoc comments on interfaces and types should be matched to the main documentation in /docs
Conformance test definitions should cover all required aspects of an FDC3 Desktop Agent implementation, which are usually marked with a MUST keyword, and optional features (SHOULD or MAY) where the format of those features is defined
The Web Connection protocol and Desktop Agent Communication Protocol schemas must be able to support all necessary aspects of the Desktop Agent API, while Bridging must support those aspects necessary for Desktop Agents to communicate with each other
npm run build) run and the results checked in?Generated code will be found at
/src/api/BrowserTypes.tsand/or/src/bridging/BridgingTypes.tsBaseContextschema applied viaallOf(as it is in existing types)?titleanddescriptionprovided for all properties defined in the schema?npm run build) run and the results checked in?Generated code will be found at
/src/context/ContextTypes.ts