insightfactory-cli 1.1.2.dev29__py3-none-any.whl → 1.1.2.dev30__py3-none-any.whl

if_cli/router/upstream.py

@@ -138,6 +138,22 @@ class ConnectUrlRecorder:
         return types.ElicitResult(action="decline")
 
 
+def _has_url_request(result: types.InputRequiredResult) -> bool:
+    """Whether any of the result's requests is the Databricks connect flow.
+
+    A request in any other mode (today, only `form`) gates a preview behind a
+    confirmation rather than asking for a trip to a browser, so its presence
+    alone never puts the result on the connect path. A URL-mode request does,
+    even one whose URL fails the origin check in `_connect_url` and so
+    contributes no link: the call still did not run, and the caller still
+    needs the existing refusal, not a result that reads as success.
+    """
+    return any(
+        isinstance(getattr(request, "params", None), types.ElicitRequestURLParams)
+        for request in (result.input_requests or {}).values()
+    )
+
+
 def _relayable_requests(result: types.InputRequiredResult, origin: str) -> tuple[list[str], str | None]:
     """The messages this router will repeat, and the first connect URL it will name.
 
@@ -279,7 +295,16 @@ class Upstream:
         result = await self._run(call, elicitation_callback=recorder)
         if isinstance(result, types.InputRequiredResult):
             lines, url = _relayable_requests(result, self.profile["host"])
-            return self._connect_refusal(lines, url)
+            if _has_url_request(result):
+                return self._connect_refusal(lines, url)
+            # Every mutating tool gates a `confirmed:false` call behind a form-mode
+            # elicitation once the router advertises elicitation at all (see
+            # `ConnectUrlRecorder`). That gate is a preview, not a connect flow: the
+            # factory's own answer to a client with no elicitation capability is a
+            # plain `pending_confirmation` result, and the router owes the caller
+            # the closest thing it can relay to that, not an error pointing at a
+            # web UI the caller was never asked to visit.
+            return self._preview_result(lines)
         if recorder.declined_url_request:
             # The older-revision path. The server elicited in band and carried on
             # once the router declined, so the connect page is in the recorder
@@ -290,6 +315,24 @@ class Upstream:
             return self._connect_refusal(_result_texts(result), recorder.url)
         return result
 
+    def _preview_result(self, lines: list[str]) -> types.CallToolResult:
+        """A form-mode approval gate, relayed as the ordinary result it is.
+
+        This loses the structured `pending_confirmation` JSON a client with no
+        elicitation capability would see (the `approvalId`, `preview`,
+        `impactedResources`): the router has only the elicitation's message
+        text to work with, not the tool's own result body, which the factory
+        never sends once it has decided to gate the call. The message still
+        carries what a caller needs, and the caller still proceeds the same
+        way, by calling again with `confirmed:true`.
+
+        Not an error: the call is still pending, exactly as it would be for a
+        client that never advertised elicitation, and marking this `is_error`
+        would make a normal preview indistinguishable from the connect refusal
+        it is not.
+        """
+        return types.CallToolResult(content=[types.TextContent(text="\n".join(lines))], is_error=False)
+
     def _connect_refusal(self, lines: list[str], url: str | None) -> types.CallToolResult:
         """Turn the factory's request for input into a refusal naming the page to visit.
 

{insightfactory_cli-1.1.2.dev29.dist-info → insightfactory_cli-1.1.2.dev30.dist-info}/METADATA

@@ -1,6 +1,6 @@
 Metadata-Version: 2.5
 Name: insightfactory-cli
-Version: 1.1.2.dev29
+Version: 1.1.2.dev30
 Summary: Profile-based authentication CLI for the InsightFactory Interfaces API
 Project-URL: Homepage, https://insightfactory.ai
 Author-email: "insightfactory.ai Support" <support@insightfactory.ai>

{insightfactory_cli-1.1.2.dev29.dist-info → insightfactory_cli-1.1.2.dev30.dist-info}/RECORD

@@ -27,9 +27,9 @@ if_cli/router/content.py,sha256=POL2n5NIY8yenQHJe8pUbTuTNpPSUSm0Y6Txbvehzhs,1448
 if_cli/router/log.py,sha256=YLWeHUb05HUnS3XHubCwy7oLZAUEGAiHcZysz_YMAAo,412
 if_cli/router/policy.py,sha256=CclbjuOA5mwwoQiNDa17iQeN9eXRbfy11bS9JFd8gL4,2840
 if_cli/router/server.py,sha256=JZ-7xbNyDadSY1VVLHpSVTjfx6vxI_uumxFaD22Pl0Y,26449
-if_cli/router/upstream.py,sha256=2V78k6SXnOJ9Epc0tnhTuUPVBrYuW5vwflT-d3xKwVc,16122
-insightfactory_cli-1.1.2.dev29.dist-info/METADATA,sha256=4M5zoiKKqT52vOoEOrx42CE_TsGyFawx1-4CLa7v5JE,23695
-insightfactory_cli-1.1.2.dev29.dist-info/WHEEL,sha256=THafob7ofN-NsuMN7Mg4qZyHaQI7KkD-QlcQatYhXPo,87
-insightfactory_cli-1.1.2.dev29.dist-info/entry_points.txt,sha256=UlX854D7f-7dBj5F0wGdakVy6vopvP9hA1BN-gv739w,44
-insightfactory_cli-1.1.2.dev29.dist-info/licenses/LICENSE,sha256=8eZ1YAABL398qESVJc_FlK1voYQ0GAh-R_rNBQK6QH8,234
-insightfactory_cli-1.1.2.dev29.dist-info/RECORD,,
+if_cli/router/upstream.py,sha256=Ga0OM918Y1CFyQFGs2wO0ocyjcMe1Wk0LvffOZOaE34,18559
+insightfactory_cli-1.1.2.dev30.dist-info/METADATA,sha256=slUXCeTP0H1dPYnOUVV2AZyWBDHlteeG3Grz5dED0Fc,23695
+insightfactory_cli-1.1.2.dev30.dist-info/WHEEL,sha256=W3fkpkm7-wf9vBI5Z-7s0eWkeM-spu78I8Neb98DeEg,87
+insightfactory_cli-1.1.2.dev30.dist-info/entry_points.txt,sha256=UlX854D7f-7dBj5F0wGdakVy6vopvP9hA1BN-gv739w,44
+insightfactory_cli-1.1.2.dev30.dist-info/licenses/LICENSE,sha256=8eZ1YAABL398qESVJc_FlK1voYQ0GAh-R_rNBQK6QH8,234
+insightfactory_cli-1.1.2.dev30.dist-info/RECORD,,

{insightfactory_cli-1.1.2.dev29.dist-info → insightfactory_cli-1.1.2.dev30.dist-info}/WHEEL

@@ -1,4 +1,4 @@
 Wheel-Version: 1.0
-Generator: hatchling 1.32.3
+Generator: hatchling 1.32.4
 Root-Is-Purelib: true
 Tag: py3-none-any