Merge remote-tracking branch 'origin/main' into mcp-ui-apps-advanced

# Conflicts:
#	ui/src/apps/issue-write/App.tsx
#	ui/src/apps/pr-write/App.tsx

Author: Copilot

ui/src/apps/issue-write/App.tsx

@@ -83,13 +83,15 @@ function SuccessView({
   submittedTitle,
   submittedLabels,
   isUpdate,
+  openLink,
 }: {
   issue: IssueResult;
   owner: string;
   repo: string;
   submittedTitle: string;
   submittedLabels: LabelItem[];
   isUpdate: boolean;
+  openLink: (url: string) => Promise<void>;
 }) {
   const issueUrl = issue.html_url || issue.url || issue.URL || "#";
 
@@ -138,6 +140,14 @@ function SuccessView({
             href={issueUrl}
             target="_blank"
             rel="noopener noreferrer"
+            onClick={(e) => {
+              // MCP Apps run in a sandboxed iframe where a plain anchor may be
+              // blocked, so route the click through the host's open-link
+              // capability (falls back to window.open).
+              e.preventDefault();
+              if (issueUrl === "#") return;
+              void openLink(issueUrl);
+            }}
             style={{
               fontWeight: 600,
               fontSize: "14px",
@@ -216,7 +226,7 @@ function CreateIssueApp() {
   const [repoSearchLoading, setRepoSearchLoading] = useState(false);
   const [repoFilter, setRepoFilter] = useState("");
 
-  const { app, error: appError, toolInput, callTool, hostContext, setModelContext } = useMcpApp({
+  const { app, error: appError, toolInput, callTool, hostContext, setModelContext, openLink } = useMcpApp({
     appName: "github-mcp-server-issue-write",
   });
 
@@ -764,6 +774,7 @@ function CreateIssueApp() {
         submittedTitle={title}
         submittedLabels={selectedLabels}
         isUpdate={isUpdateMode}
+        openLink={openLink}
       />
     );
   }

ui/src/apps/pr-write/App.tsx

@@ -52,11 +52,13 @@ function SuccessView({
   owner,
   repo,
   submittedTitle,
+  openLink,
 }: {
   pr: PRResult;
   owner: string;
   repo: string;
   submittedTitle: string;
+  openLink: (url: string) => Promise<void>;
 }) {
   const prUrl = pr.html_url || pr.url || pr.URL || "#";
 
@@ -105,6 +107,14 @@ function SuccessView({
             href={prUrl}
             target="_blank"
             rel="noopener noreferrer"
+            onClick={(e) => {
+              // MCP Apps run in a sandboxed iframe where a plain anchor may be
+              // blocked, so route the click through the host's open-link
+              // capability (falls back to window.open).
+              e.preventDefault();
+              if (prUrl === "#") return;
+              void openLink(prUrl);
+            }}
             style={{
               fontWeight: 600,
               fontSize: "14px",
@@ -157,7 +167,7 @@ function CreatePRApp() {
   const [repoSearchLoading, setRepoSearchLoading] = useState(false);
   const [repoFilter, setRepoFilter] = useState("");
 
-  const { app, error: appError, toolInput, callTool, hostContext, setModelContext } = useMcpApp({
+  const { app, error: appError, toolInput, callTool, hostContext, setModelContext, openLink } = useMcpApp({
     appName: "github-mcp-server-create-pull-request",
   });
 
@@ -352,7 +362,7 @@ function CreatePRApp() {
   if (successPR) {
     return (
       <AppProvider hostContext={hostContext}>
-        <SuccessView pr={successPR} owner={owner} repo={repo} submittedTitle={submittedTitle} />
+        <SuccessView pr={successPR} owner={owner} repo={repo} submittedTitle={submittedTitle} openLink={openLink} />
       </AppProvider>
     );
   }

ui/src/hooks/useMcpApp.ts

@@ -106,7 +106,12 @@ export function useMcpApp({
         window.open(url, "_blank", "noopener,noreferrer");
         return;
       }
-      await app.openLink({ url });
+      const result = await app.openLink({ url });
+      // The host may deny the request (e.g. blocked domain or user cancelled).
+      // Fall back to a direct window.open so the link still works.
+      if (result?.isError) {
+        window.open(url, "_blank", "noopener,noreferrer");
+      }
     },
     [app]
   );