Skip to content

docs(bootstrap): clarify trace step comment, drop stale libghostty note#35

Closed
eouzoe wants to merge 1 commit into
mainfrom
pr/startsh-trace-comment
Closed

docs(bootstrap): clarify trace step comment, drop stale libghostty note#35
eouzoe wants to merge 1 commit into
mainfrom
pr/startsh-trace-comment

Conversation

@eouzoe

@eouzoe eouzoe commented Jul 16, 2026

Copy link
Copy Markdown
Owner

修改了什麼

scripts/start.sh 第 5 步(devenv 啟動 + trace)的註釋清理:移除過時的 (zsh via libghostty) 措辭,並補充 trace 機制的說明(用 devenv 原生 --trace-to,不強制 OTLP 導出)。

為什麼要修改

  • libghostty 是 devenv 2.1 內部實作細節,不該寫在引導腳本的面向使用者註釋裡,且易與「WSL/終端」混淆。
  • 需明確表達:trace 只走 --trace-to pretty:stderr,是否導出 OTLP 由使用者自行決定,腳本不硬塞 grpc 導出。

功能

  • 無行為變更,純註釋。

涉及檔案

  • scripts/start.sh(註釋 +4 / -1,第 5 步)

影響

參考文件 / 依據

  • devenv 2.1 官方文件:--trace-to 統一 flag,pretty:stderr 為 stderr 漂亮輸出;otlp-grpc / otlp-http-protobuf 才導出。本機 devenv 2.1.2 --help 已確認 pretty:stderr 為受支持語法。
  • 標的環境為 NixOS-WSL,devenv 經 nix profile add nixpkgs#devenv 安裝,trace flag 同樣適用。

測試

  • bash -n scripts/start.sh 語法通過。
  • 本機 devenv 2.1.2 實測 devenv --trace-to pretty:stderr --help 接受該 flag(exit 0)。

@eouzoe eouzoe closed this Jul 16, 2026
@eouzoe
eouzoe deleted the pr/startsh-trace-comment branch July 16, 2026 20:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant