document: ブラウザで書類を開くopenサブコマンドを追加 (#21)

既定のブラウザで書類の表示ページを開く `xp document open <docid>`
を追加。--no-browser (-n) で URL のみを出力する。

Co-authored-by: Claude Opus 4.7 <noreply@anthropic.com>

Author: buty4649

README.md

@@ -105,6 +105,13 @@ xp document status 266248 --history         # 全バージョンの承認履歴
 xp document status 266248 --jq '.document.status.name'
 ```
 
+### ドキュメントをブラウザで開く
+
+```sh
+xp document open 266248              # 既定のブラウザで書類を開く
+xp document open 266248 --no-browser # URLだけ出力（ブラウザは起動しない）
+```
+
 ### ドキュメントのPDFダウンロード
 
 ```sh

cmd/document.go

@@ -41,6 +41,8 @@ var (
 
 	docStatusHistory bool
 	docStatusJQ      string
+
+	docOpenNoBrowser bool
 )
 
 var documentCmd = &cobra.Command{
@@ -127,6 +129,17 @@ approval histories for all past versions.`,
 	RunE: runDocumentStatus,
 }
 
+var documentOpenCmd = &cobra.Command{
+	Use:   "open <docid>",
+	Short: "Open a document in the default web browser",
+	Long: `Open the document view page in the default web browser.
+
+The URL is built from the configured subdomain (no API request is made).
+Pass --no-browser (or -n) to print the URL without launching the browser.`,
+	Args: cobra.ExactArgs(1),
+	RunE: runDocumentOpen,
+}
+
 var documentDownloadCmd = &cobra.Command{
 	Use:   "download <docid>",
 	Short: "Download a document as PDF",
@@ -150,6 +163,7 @@ func init() {
 	documentCmd.AddCommand(documentDeleteCmd)
 	documentCmd.AddCommand(documentStatusCmd)
 	documentCmd.AddCommand(documentDownloadCmd)
+	documentCmd.AddCommand(documentOpenCmd)
 
 	f := documentSearchCmd.Flags()
 	f.StringVar(&docSearchBody, "body", "", "search condition JSON: inline, file path, or - for stdin")
@@ -184,6 +198,9 @@ func init() {
 
 	dlf := documentDownloadCmd.Flags()
 	dlf.StringVarP(&docDownloadOutput, "output", "o", "", "output path: FILE, DIR/, or - for stdout (default: server-provided filename in current directory)")
+
+	of := documentOpenCmd.Flags()
+	of.BoolVarP(&docOpenNoBrowser, "no-browser", "n", false, "print the URL without launching the browser")
 }
 
 func runDocumentSearch(cmd *cobra.Command, args []string) error {
@@ -401,6 +418,28 @@ func runDocumentDownload(cmd *cobra.Command, args []string) error {
 	return nil
 }
 
+func runDocumentOpen(_ *cobra.Command, args []string) error {
+	docID, err := parseDocID(args[0])
+	if err != nil {
+		return err
+	}
+	sub, err := resolveSubdomain()
+	if err != nil {
+		return err
+	}
+	url := xpoint.NewClient(sub, xpoint.Auth{}).DocumentURL(docID)
+
+	if docOpenNoBrowser {
+		fmt.Fprintln(os.Stdout, url)
+		return nil
+	}
+	fmt.Fprintf(os.Stdout, "Opening %s\n", url)
+	if err := openBrowser(url); err != nil {
+		return fmt.Errorf("open browser: %w (run with --no-browser to print the URL)", err)
+	}
+	return nil
+}
+
 // resolveDownloadPath decides the on-disk path for a downloaded PDF.
 //
 // When out is empty, the server-provided filename is used in the current

cmd/document_test.go

@@ -167,6 +167,43 @@ func TestRunDocumentEdit_RequiresBody(t *testing.T) {
 	}
 }
 
+func TestRunDocumentOpen_InvalidDocID(t *testing.T) {
+	err := runDocumentOpen(documentOpenCmd, []string{"0"})
+	if err == nil || !strings.Contains(err.Error(), "invalid docid") {
+		t.Errorf("err = %v", err)
+	}
+}
+
+func TestRunDocumentOpen_NoBrowserPrintsURL(t *testing.T) {
+	docOpenNoBrowser = true
+	t.Cleanup(func() { docOpenNoBrowser = false })
+	t.Setenv("XPOINT_SUBDOMAIN", "acme")
+
+	origStdout := os.Stdout
+	r, w, err := os.Pipe()
+	if err != nil {
+		t.Fatalf("pipe: %v", err)
+	}
+	os.Stdout = w
+	t.Cleanup(func() { os.Stdout = origStdout })
+
+	done := make(chan string, 1)
+	go func() {
+		b, _ := io.ReadAll(r)
+		done <- string(b)
+	}()
+
+	if err := runDocumentOpen(documentOpenCmd, []string{"266248"}); err != nil {
+		t.Fatalf("runDocumentOpen: %v", err)
+	}
+	_ = w.Close()
+	got := strings.TrimSpace(<-done)
+	want := "https://acme.atledcloud.jp/xpoint/form.do?act=view&docid=266248"
+	if got != want {
+		t.Errorf("stdout = %q, want %q", got, want)
+	}
+}
+
 func TestRunDocumentEdit_InvalidDocID(t *testing.T) {
 	err := runDocumentEdit(documentEditCmd, []string{"not-a-number"})
 	if err == nil || !strings.Contains(err.Error(), "invalid docid") {