Merge branch 'main' of https://github.com/Azure/LogicAppsUX into ccastrotrejo/iframeExcludeURls

Author: ccastrotrejo

.github/workflows/pr-coverage.yml

@@ -89,6 +89,9 @@ jobs:
             **/types.ts
             **/types.tsx
             **/styles.ts
+            **/styles.tsx
+            **/*.styles.ts
+            **/*.styles.tsx
             **/constants.ts
             **/constants/**
             # Config files
@@ -128,6 +131,8 @@ jobs:
             **/*Provider.ts
             **/*Provider.tsx
             **/providers/**
+            # VS Code webview communication (uses acquireVsCodeApi runtime global, cannot be unit tested)
+            **/webviewCommunication.tsx
 
       - name: Check coverage on changed files
         id: coverage-check
@@ -141,6 +146,7 @@ jobs:
           if [ -z "$CHANGED_FILES" ]; then
             echo "No source files changed in this PR." >> $GITHUB_STEP_SUMMARY
             echo "coverage_status=skip" >> $GITHUB_OUTPUT
+            printf '%s\n\n%s\n' "## 📊 Coverage Check" "No source files changed in this PR." > ./coverage-comment.md
             exit 0
           fi
 
@@ -161,6 +167,7 @@ jobs:
               echo "" >> $GITHUB_STEP_SUMMARY
               echo "❌ **No coverage data found.** Please ensure tests generate coverage reports." >> $GITHUB_STEP_SUMMARY
               echo "coverage_status=no_data" >> $GITHUB_OUTPUT
+              printf '%s\n\n%s\n' "## 📊 Coverage Check" "⚠️ No coverage data found. Please ensure tests generate coverage reports." > ./coverage-comment.md
               exit 0
             fi
           fi
@@ -265,6 +272,27 @@ jobs:
             echo "coverage_status=pass" >> $GITHUB_OUTPUT
           fi
 
+          # Build a concise PR comment with failing files
+          {
+            echo "## 📊 Coverage Check"
+            echo ""
+            if [ -n "$UNCOVERED_FILES" ] || [ -n "$PARTIAL_FILES" ]; then
+              echo "The following changed files need attention:"
+              echo ""
+              if [ -n "$UNCOVERED_FILES" ]; then
+                echo -e "$UNCOVERED_FILES"
+                echo ""
+              fi
+              if [ -n "$PARTIAL_FILES" ]; then
+                echo -e "$PARTIAL_FILES"
+                echo ""
+              fi
+              echo "**Please add tests for the uncovered files before merging.**"
+            else
+              echo "🎉 **All changed files have adequate test coverage!**"
+            fi
+          } > ./coverage-comment.md
+
       - name: Post coverage comment on PR
         if: always()
         uses: actions/github-script@v7
@@ -281,17 +309,18 @@ jobs:
               return;
             }
 
-            // Read the step summary
-            const summaryPath = process.env.GITHUB_STEP_SUMMARY;
+            const COMMENT_MARKER = '<!-- pr-coverage-check-comment -->';
+
+            // Read the coverage comment built by the coverage-check step
             let summary = '';
             try {
-              summary = fs.readFileSync(summaryPath, 'utf8');
+              summary = fs.readFileSync('./coverage-comment.md', 'utf8');
             } catch (e) {
               summary = '📊 Coverage check completed. See workflow run for details.';
             }
 
             try {
-              // Find existing comment
+              // Find existing comment by hidden marker
               const { data: comments } = await github.rest.issues.listComments({
                 owner: context.repo.owner,
                 repo: context.repo.repo,
@@ -300,10 +329,10 @@ jobs:
 
               const botComment = comments.find(comment =>
                 comment.user.type === 'Bot' &&
-                comment.body.includes('📊 Coverage Report for Changed Files')
+                comment.body.includes(COMMENT_MARKER)
               );
 
-              const commentBody = summary || '📊 Coverage check completed. See workflow run for details.';
+              const commentBody = COMMENT_MARKER + '\n' + (summary || '📊 Coverage check completed. See workflow run for details.');
 
               if (botComment) {
                 await github.rest.issues.updateComment({

Localize/lang/strings.json

@@ -253,6 +253,7 @@
   "3n5pnv": "(UTC+10:30) Lord Howe Island",
   "3n6GJG": "Enter a valid JSON.",
   "3nUf86": "Provided value",
+  "3nbg4r": "From",
   "3pOMqH": "This could mean that the variable is set up incorrectly.",
   "3pheF6": "Select schema",
   "3qDalD": "Add optional parameters",
@@ -590,6 +591,7 @@
   "BKL0ZG": "Start time",
   "BMCFV3": "If available, default parameters specify the minimum inputs for this task. You can add any optional parameters for your scenario. By default, parameters get dynamic values from the AI model, but you can choose to enter static values instead.",
   "BO1cXH": "Either limit count or timeout must be specified.",
+  "BOD0+G": "To",
   "BP+WUL": "Close",
   "BQCPY7": "Required. The string to decode the URL-unsafe characters from.",
   "BQSRV0": "Enter password as plain text or use a secure parameter",
@@ -878,6 +880,7 @@
   "IQyOth": "If available, dynamic content is automatically generated from the connectors and actions you choose for your flow.",
   "IS4vNX": "(UTC-12:00) International Date Line West",
   "ISaPr+": "Create, manage Logic Apps parameters, give it a default value.",
+  "IU8Zu9": "Select date",
   "IUbVFR": "Search",
   "IW2MjQ": "Converts a string timestamp passed in from a UTC to a target time zone",
   "IXy91L": "Add a target schema",
@@ -1167,6 +1170,7 @@
   "PNk3n4": "Runs {onTime}",
   "PORNMZ": "Inputs",
   "PP63jY": "more panels",
+  "PPssRb": "Custom range",
   "PPvsfZ": "No supported channels exist for this agent.",
   "PQOiAc": "Collapse",
   "PRaFHG": "Workflow is in progress",
@@ -1406,6 +1410,7 @@
   "Ug4sWZ": "Expand",
   "UgaIRz": "Required. The length of each chunk.",
   "UksFS1": "Select agents",
+  "Ul4k7r": "Last 48 hours",
   "Umpr3z": "Every {interval} weeks",
   "Unc2tG": "Returns true if two values are equal.",
   "UnrrzF": "Source schema",
@@ -1465,6 +1470,7 @@
   "W5bj8F": "Description",
   "W621ZA": "Contact info",
   "W6FdMh": "Required. The name of the new property.",
+  "W7l+j+": "Last 14 days",
   "W83QYZ": "No results for",
   "W99jiu": "Show description",
   "WBDuOo": "Fetching...",
@@ -1505,6 +1511,7 @@
   "WxcmZr": "This action has testing configured.",
   "WyH1wr": "Searching for results...",
   "X/7je+": "Minute",
+  "X0/aJy": "Last 30 days",
   "X02GGK": "Tags",
   "X1TOAH": "Enter operation description",
   "X2idLs": "(UTC-03:00) Montevideo",
@@ -1904,6 +1911,7 @@
   "_3n5pnv.comment": "Time zone value ",
   "_3n6GJG.comment": "Error validation message for Objects",
   "_3nUf86.comment": "Label for input field in parameter editor",
+  "_3nbg4r.comment": "Custom time range start label",
   "_3pOMqH.comment": "Warning body for when unable to parse variables",
   "_3pheF6.comment": "Select schema",
   "_3qDalD.comment": "Label for adding new optional parameters in the dropdown",
@@ -2241,6 +2249,7 @@
   "_BKL0ZG.comment": "Column header text for start time",
   "_BMCFV3.comment": "Description for the parameters section",
   "_BO1cXH.comment": "Error message to show when either limit count or timeout is not specified.",
+  "_BOD0+G.comment": "Custom time range end label",
   "_BP+WUL.comment": "Button text for exiting the blade in the clone wizard",
   "_BQCPY7.comment": "Required string parameter to be decoded using decodeUriComponent function",
   "_BQSRV0.comment": "Basic Password Placeholder Text",
@@ -2529,6 +2538,7 @@
   "_IQyOth.comment": "Section 1 of text for including dynamic content section",
   "_IS4vNX.comment": "Time zone value ",
   "_ISaPr+.comment": "Description for Workflow Parameters Part 1 for Legacy Parameters mode.",
+  "_IU8Zu9.comment": "Placeholder for date picker input",
   "_IUbVFR.comment": "Placeholder text for search templates",
   "_IW2MjQ.comment": "Label for description of custom convertFromUtc Function",
   "_IXy91L.comment": "label to inform to add a target schema to be used",
@@ -2818,6 +2828,7 @@
   "_PNk3n4.comment": "Recurrence schedule description on days of week at times",
   "_PORNMZ.comment": "Inputs text",
   "_PP63jY.comment": "This is a label to access the overflowed panels",
+  "_PPssRb.comment": "Custom time range filter",
   "_PPvsfZ.comment": "Channel not supported message",
   "_PQOiAc.comment": "An accessible label for collapse toggle icon",
   "_PRaFHG.comment": "Text shown when a run is in progress",
@@ -3057,6 +3068,7 @@
   "_Ug4sWZ.comment": "Expand to make the node bigger and show the contents.",
   "_UgaIRz.comment": "Required number parameter to get length of each chunk for chunk function",
   "_UksFS1.comment": "Select Agents",
+  "_Ul4k7r.comment": "Last 48 hours filter",
   "_Umpr3z.comment": "Recurrence schedule description every interval weeks",
   "_Unc2tG.comment": "Label for description of custom equals Function",
   "_UnrrzF.comment": "Label to inform the below schema name is for source schema",
@@ -3116,6 +3128,7 @@
   "_W5bj8F.comment": "Label for the MCP Server description field",
   "_W621ZA.comment": "Contact info section title",
   "_W6FdMh.comment": "Required string parameter for new property name in addProperty function",
+  "_W7l+j+.comment": "Last 14 days filter",
   "_W83QYZ.comment": "Text displayed when no results match the search term",
   "_W99jiu.comment": "Toggle button label to show comment section",
   "_WBDuOo.comment": "Fetching data text",
@@ -3156,6 +3169,7 @@
   "_WxcmZr.comment": "This is a tooltip for the Status results badge shown on a card. It's shown when the baged is hovered over.",
   "_WyH1wr.comment": "Message to show when loading search results",
   "_X/7je+.comment": "Frequency value ",
+  "_X0/aJy.comment": "Last 30 days filter",
   "_X02GGK.comment": "Title for the tags section in the template overview tab",
   "_X1TOAH.comment": "Placeholder text for operation description field",
   "_X2idLs.comment": "Time zone value ",
@@ -3809,6 +3823,7 @@
   "_lbq5E1.comment": "description of upload content transfer setting",
   "_lciYKh.comment": "Loading message for the MCP server workflows field",
   "_lckgnb.comment": "Required collection parameter to apply reverse function on",
+  "_ldBi4y.comment": "Aria label for the toggle filters button",
   "_ldn/IC.comment": "Test button",
   "_leYJf/.comment": "Description for workflow display name field",
   "_lexzgJ.comment": "Message on failure to remove source node",
@@ -3958,6 +3973,7 @@
   "_odQ554.comment": "Response body for test map API",
   "_oeQQ8q.comment": "Label for array item",
   "_og5JOA.comment": "Millisecond",
+  "_oglOen.comment": "Time interval filter label",
   "_ohpbkw.comment": "title for retry policy exponential interval setting",
   "_ol3TWp.comment": "Button label to automaticlaly generate agent parameter",
   "_olgoo5.comment": "Title for step 2 - tools selection",
@@ -4074,6 +4090,7 @@
   "_rSIBjh.comment": "Parameter Field Value Placeholder Text",
   "_rSa1Id.comment": "Files could not be found in specified path",
   "_raBiud.comment": "Require parameters to find maximum using max function",
+  "_rbVW+z.comment": "Last 24 hours filter",
   "_rcz4w4.comment": "Label for description of custom uriComponent Function",
   "_rd6fai.comment": "Aria describing the way to control the keyboard navigation",
   "_rh5g4p.comment": "Successful run",
@@ -4109,6 +4126,7 @@
   "_sRpETS.comment": "Warning message for when custom value does not match schema node type",
   "_sVQe34.comment": "The description for the test tab parameters.",
   "_sVcvcG.comment": "The tab label for the monitoring name and state tab on the create workflow panel",
+  "_sXFMqZ.comment": "All time intervals",
   "_sYQDN+.comment": "Label for Font family dropdown",
   "_sZ0G/Z.comment": "Required string parameter to represent the unit of time",
   "_sZHTQV.comment": "Time zone value ",
@@ -4128,6 +4146,7 @@
   "_snzCiK.comment": "Label to hide password",
   "_soqP+Z.comment": "Label for description of custom or Function",
   "_sqA07R.comment": "Button text for moving to the previous tab in the connector panel",
+  "_sqeFva.comment": "Placeholder for time picker input",
   "_sqsHMO.comment": "Label for the Azure Container Apps session pool",
   "_srMbm9.comment": "Chatbot undo operation cancel button text",
   "_srg0hY.comment": "Label text for production publish state",
@@ -4263,6 +4282,7 @@
   "_v3Xc8p.comment": "Label for description of custom decodeXmlValue Function",
   "_v5CBNu.comment": "Default value label",
   "_v6V2NA.comment": "Text for the \"Deselect All\" option in a multiselect dropdown",
+  "_v80mnx.comment": "Last 7 days filter",
   "_v95bFR.comment": "Error message title for duplicate workflow ids",
   "_vAdBMk.comment": "Next button text",
   "_vAtGzU.comment": "Path to the file to select",
@@ -4983,6 +5003,7 @@
   "lbq5E1": "Large messages may be split up into smaller requests to the connector to allow large message upload. More details can be found at http://aka.ms/logicapps-chunk#upload-content-in-chunks",
   "lciYKh": "Loading workflows...",
   "lckgnb": "Required. The collection to reverse.",
+  "ldBi4y": "Toggle filters",
   "ldn/IC": "Test",
   "leYJf/": "The unique internal system name for this workflow. Use lowercase letters, numbers, and hyphens only—no spaces or special characters.",
   "lexzgJ": "Remove all references to element ' {nodeName} ' before you remove the element.",
@@ -5132,6 +5153,7 @@
   "odQ554": "Response body",
   "oeQQ8q": "{label} item",
   "og5JOA": "{count} Millisecond",
+  "oglOen": "Time range",
   "ohpbkw": "Exponential interval",
   "ol3TWp": "Select to generate the agent parameter",
   "olgoo5": "Select allowed tools",
@@ -5248,6 +5270,7 @@
   "rSIBjh": "Enter value for parameter.",
   "rSa1Id": "No files found in {filePath}, please save XSLT to specified path to use this function",
   "raBiud": "Required. Either an array of values to find the maximum value, or the first value of a set.",
+  "rbVW+z": "Last 24 hours",
   "rcz4w4": "Returns a URI encoded representation of a value",
   "rd6fai": "Use left and right arrow keys to navigate between commands",
   "rh5g4p": "Is successful",
@@ -5283,6 +5306,7 @@
   "sRpETS": "Warning: custom value does not match the schema node's type",
   "sVQe34": "Provide parameters to test the output.",
   "sVcvcG": "Basics",
+  "sXFMqZ": "All",
   "sYQDN+": "Formatting options for font family",
   "sZ0G/Z": "Required. A string containing the unit of time specified in the interval to add.",
   "sZHTQV": "(UTC+09:00) Chita",
@@ -5302,6 +5326,7 @@
   "snzCiK": "Hide Password",
   "soqP+Z": "Returns true if either parameter is true",
   "sqA07R": "Previous",
+  "sqeFva": "Select time",
   "sqsHMO": "Session Pool",
   "srMbm9": "Cancel",
   "srg0hY": "Production",
@@ -5437,6 +5462,7 @@
   "v3Xc8p": "Decodes a string that has been encoded as an XML element value back to its original form.",
   "v5CBNu": "Default value",
   "v6V2NA": "Deselect all",
+  "v80mnx": "Last 7 days",
   "v95bFR": "Workflow names must be unique. Duplicate workflow ids:",
   "vAdBMk": "Next",
   "vAtGzU": "Select file",