openclacky 1.0.0.beta.1 → 1.0.0.beta.2

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: b0938f51d788566d1c708a0054a3b45565f0f3b2ea7cd68f9e18897247c745cd
-  data.tar.gz: 96bef1e4b6333fcee57e26fb330f12d7d5b9f79f470a34f35483ab5f51571e64
+  metadata.gz: a6a51d00de51f04f142d6be7e3e726561384752ca02f44a6971eeaad9c2cdb28
+  data.tar.gz: 185e635750793082206332377649095313b62297a2bd2f9f0b825a523a499afb
 SHA512:
-  metadata.gz: 59ba5927fef6187d4d862f210282190b589838e4950b0a5bf0e37731e6cd772bba0d4bfd483d6ff14f979eabf7a4dc09abacb2c60f98df7762af6b4191dc67cf
-  data.tar.gz: 52063a91b64281d4dcf3e6bd8e3f4d87cc63cfed70169c87cfacf55e4f6c1cb16f2d938a9f554f564829e879755573d3eb8fb3988260285d11087dfa0058473d
+  metadata.gz: 66e14242f2d4b0e049fd45283c77c71919614d5f2ef8558b8635318f443afd41ab3ebf941a3b10a22c987805569082f699683af5eae23f5ef54e6dbd993e4931
+  data.tar.gz: 4734d321c296e168f505185e67bddf8967687fd4889871a9a148f8bc003a7bb866774eb4f8c15ab4aba8b27384ae28e1c436477b3dd65e898c90743b393dbfb9

data/.clacky/skills/gem-release/SKILL.md

@@ -3,7 +3,9 @@
 name: gem-release
 description: >-
   Automates the complete process of releasing a new version of the openclacky Ruby
-  gem
+  gem. Supports both stable releases (auto-increment) and pre-release versions
+  (user-specified, e.g., 1.0.0.beta.1). Handles version bumping, testing, building,
+  RubyGems publishing, GitHub Releases, and OSS CDN mirroring.
 disable-model-invocation: false
 user-invocable: true
 ---
@@ -21,6 +23,7 @@ This skill handles the entire gem release workflow from version bumping to publi
 To use this skill, simply say:
 - "Release a new version"
 - "Publish a new gem version"
+- "Release version 1.0.0.beta.1" (pre-release with explicit version)
 - Use the command: `/gem-release`
 
 ## Process Steps
@@ -31,10 +34,30 @@ To use this skill, simply say:
 - Ensure the repository is in a clean state
 
 ### 2. Version Management
+
+**Stable releases (default):**
 - Read current version from `lib/clacky/version.rb`
 - Increment version number (typically patch version: x.y.z → x.y.z+1)
 - Update the VERSION constant in the version file
 
+**Pre-release versions (when user specifies a version like `1.0.0.beta.1`):**
+- Accept the user-provided version string directly — do NOT auto-increment
+- The version must follow semver pre-release format: `X.Y.Z-<identifier>` or `X.Y.Z.<identifier>` (e.g., `1.0.0.beta.1`, `2.0.0-alpha`, `1.5.0-rc1`)
+- Before proceeding, warn the user about pre-release caveats (see Pre-Release Caveats below)
+
+### 2a. Pre-Release Caveats
+
+When releasing a pre-release version, inform the user of these known behaviors in the Clacky ecosystem:
+
+| Concern | Behavior | Impact |
+|---------|----------|--------|
+| **Version check notification** | RubyGems API returns the highest version number, including prereleases. `Gem::Version("0.9.38") < Gem::Version("1.0.0.beta.1")` → `true`. | ✅ The upgrade dot WILL appear in the Web UI for most users. |
+| **`gem update` (official source)** | `gem update openclacky --no-document` does NOT install prereleases without `--pre`. | ❌ Users on official RubyGems source who click "Upgrade" will see the notification but the upgrade will silently do nothing. |
+| **OSS CDN upgrade (mirror users)** | `upgrade_via_oss_cdn` downloads the exact `.gem` from `latest.txt` on OSS. | ⚠️ If you update `latest.txt` to point to the prerelease, mirror users WILL get the beta. |
+| **OSS `latest.txt`** | Stable users fetching `latest.txt` for fresh installs would get the beta. | ⚠️ By default, do NOT update `latest.txt` for pre-releases. Only update if this is intentional (e.g., a release candidate for broad testing). |
+
+**Action**: Ask the user whether to update `latest.txt` on OSS before proceeding. For internal testing, the answer is usually "no".
+
 ### 3. Quality Assurance
 - Run the full test suite with `bundle exec rspec`
 - Ensure all 167+ tests pass
@@ -93,15 +116,26 @@ To use this skill, simply say:
 
 4. **Create GitHub Release and Upload gem**
 
-   Extract the release notes for this version from CHANGELOG.md, then create a GitHub Release with the .gem file attached:
+   Extract the release notes for this version from CHANGELOG.md, then create a GitHub Release with the .gem file attached.
+
+   **For stable releases:**
    ```bash
    gh release create v{version} \
      --title "v{version}" \
-     --notes-file /tmp/release_notes.md \
+     --notes-file /tmp/release_notes_{version}.md \
      --latest \
      openclacky-{version}.gem
    ```
 
+   **For pre-release versions (e.g., `1.0.0.beta.1`):** use `--prerelease` instead of `--latest`:
+   ```bash
+   gh release create v{version} \
+     --title "v{version}" \
+     --notes-file /tmp/release_notes_{version}.md \
+     --prerelease \
+     openclacky-{version}.gem
+   ```
+
    Steps:
    - Parse the CHANGELOG.md section for `[{version}]`
    - Write it to a temp file (e.g., `/tmp/release_notes_{version}.md`) to avoid shell escaping issues
@@ -112,22 +146,28 @@ To use this skill, simply say:
 
 5. **Sync to Tencent Cloud OSS (CN mirror)**
 
-   After GitHub Release is created, upload the .gem file and update `latest.txt` on OSS so Chinese users can install without hitting GitHub directly:
+   After GitHub Release is created, upload the .gem file to OSS so Chinese users can install without hitting GitHub directly.
 
    ```bash
-   # Upload .gem file
+   # Upload .gem file (always do this for any release)
    coscli cp openclacky-{version}.gem cos://clackyai-1258723534/openclacky/openclacky-{version}.gem
+   ```
 
-   # Update latest.txt
+   **For stable releases only** — update `latest.txt` so fresh installs and mirror users pick up the new version:
+   ```bash
    echo "{version}" > /tmp/latest.txt
    coscli cp /tmp/latest.txt cos://clackyai-1258723534/openclacky/latest.txt
 
    # Verify
    curl -fsSL https://oss.1024code.com/openclacky/latest.txt
    ```
-
    Expected output of verify: `{version}`
 
+   **For pre-release versions** — do NOT update `latest.txt` unless the user explicitly requested it. Updating `latest.txt` to a prerelease would cause:
+   - Mirror users clicking "Upgrade" to get the beta via `upgrade_via_oss_cdn`
+   - Fresh installs via the install script to get the beta
+   - Only skip this if the user explicitly wants broad beta distribution
+
    > **Prerequisite**: `coscli` installed at `/usr/local/bin/coscli` and configured at `~/.cos.yaml`
 
 6. **Sync scripts/ to OSS**
@@ -325,22 +365,34 @@ git tag vX.Y.Z
 git push origin main
 git push origin --tags
 
-# Create GitHub Release with .gem asset (requires gh CLI)
-# 1. Extract release notes from CHANGELOG.md for this version
-# 2. Write to temp file to avoid shell escaping issues
-# 3. Create the release and attach .gem file
+# ── GitHub Release ──────────────────────────────────────────────────────
+
+# Stable release:
 gh release create vX.Y.Z \
   --title "vX.Y.Z" \
   --notes-file /tmp/release_notes_X.Y.Z.md \
   --latest \
   openclacky-X.Y.Z.gem
 
-# Sync to Tencent Cloud OSS (CN mirror)
+# Pre-release (use --prerelease instead of --latest):
+gh release create vX.Y.Z-beta.1 \
+  --title "vX.Y.Z-beta.1" \
+  --notes-file /tmp/release_notes_X.Y.Z-beta.1.md \
+  --prerelease \
+  openclacky-X.Y.Z.beta.1.gem
+
+# ── OSS CDN (CN mirror) ─────────────────────────────────────────────────
+
+# Always upload the .gem file:
 coscli cp openclacky-X.Y.Z.gem cos://clackyai-1258723534/openclacky/openclacky-X.Y.Z.gem
+
+# Stable releases ONLY — update latest.txt:
 echo "X.Y.Z" > /tmp/latest.txt
 coscli cp /tmp/latest.txt cos://clackyai-1258723534/openclacky/latest.txt
 curl -fsSL https://oss.1024code.com/openclacky/latest.txt  # verify
 
+# Pre-releases — skip latest.txt update unless user explicitly requests it
+
 # Sync scripts/ to OSS (build from templates first)
 bash scripts/build/build.sh
 for script in scripts/*; do
@@ -365,8 +417,10 @@ curl -fsSL https://oss.1024code.com/clacky-ai/openclacky/main/scripts/install.sh
 - Git repository updated with version tag
 - CHANGELOG.md updated with release notes
 - GitHub Release created with .gem file attached at https://github.com/clacky-ai/openclacky/releases
+  - Use `--latest` for stable releases, `--prerelease` for pre-releases
 - .gem file uploaded to OSS: https://oss.1024code.com/openclacky/openclacky-{version}.gem
-- latest.txt updated on OSS: https://oss.1024code.com/openclacky/latest.txt returns the new version
+- For stable releases: `latest.txt` updated on OSS: https://oss.1024code.com/openclacky/latest.txt returns the new version
+- For pre-releases: `latest.txt` NOT updated (unless user explicitly opts in)
 - No build or deployment errors
 - User-facing release summary presented at the end
 

data/CHANGELOG.md

@@ -7,6 +7,16 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
 
 ## [Unreleased]
 
+## [1.0.0.beta.2] - 2026-04-27
+
+### Added
+- **New session creation supports model & working-directory options.** The Web UI "new session" dialog now lets you pick the model and starting directory up front, instead of having to adjust them after the session opens.
+
+### Fixed
+- **System prompt now refreshes when you switch models.** Previously the system prompt captured at session start stuck around even after `/model` or `/provider` switches, which could leave model-specific instructions out of sync. The agent now re-injects the correct system prompt on every model change.
+- **Port 7070 properly released when the terminal tool exits.** A lingering listener on port 7070 could block subsequent runs; the terminal tool now cleans it up on shutdown.
+- **Windows installer uses `[IO.Path]::GetTempPath()` for the temp directory** (#58) — more reliable than `$env:TEMP` on systems where the env var is unset or points to a non-ASCII path.
+
 ## [1.0.0.beta.1] - 2026-04-26
 
 ### Added

data/lib/clacky/agent.rb

@@ -1371,7 +1371,19 @@ module Clacky
     # Core method to inject session context (date, model, OS, paths).
     # Called by inject_session_context_if_needed (with date check)
     # and by switch_model (without date check, to force update).
+    #
+    # IMPORTANT: Skip injection when the system prompt hasn't been built yet.
+    # Otherwise, appending a user message to an empty history makes
+    # @history.empty? false, which causes run() to skip building the
+    # system prompt entirely (see run()'s "first run" guard).
+    # The injection will happen naturally in run() via
+    # inject_session_context_if_needed after the system prompt is in place.
     private def inject_session_context
+      # Don't inject context before system prompt exists — defer to
+      # inject_session_context_if_needed which runs inside run()
+      # after the system prompt has been built.
+      return unless @history.has_system_prompt?
+
       today   = Time.now.strftime("%Y-%m-%d")
       os      = Clacky::Utils::EnvironmentDetector.os_type
       desktop = Clacky::Utils::EnvironmentDetector.desktop_path

data/lib/clacky/message_history.rb

@@ -100,6 +100,14 @@ module Clacky
     # Business queries
     # ─────────────────────────────────────────────
 
+    # True when a system prompt message is present in the history.
+    # Used by inject_session_context to avoid injecting context messages
+    # before the system prompt has been built (which would cause the
+    # guard in run() to skip building it altogether).
+    def has_system_prompt?
+      @messages.any? { |m| m[:role] == "system" }
+    end
+
     # True when the last assistant message has tool_calls but no
     # tool_result has been appended yet (would cause a 400 from the API).
     def pending_tool_calls?

data/lib/clacky/server/http_server.rb

@@ -501,15 +501,25 @@ module Clacky
         raw_dir = body["working_dir"].to_s.strip
         working_dir = raw_dir.empty? ? default_working_dir : File.expand_path(raw_dir)
 
-        # Optional model override
-        model_override = body["model"].to_s.strip
-        model_override = nil if model_override.empty?
+        # Optional model override — passed as a stable model id (matches the
+        # id returned by GET /api/config). Name-based override was removed:
+        # a bare model name can't disambiguate between entries from different
+        # providers (e.g. "deepseek-v4-pro" on DeepSeek direct vs its dsk-*
+        # alias on OpenClacky/Bedrock), and mutating current_model["model"]
+        # kept the wrong api_key / base_url / api format, producing
+        # "unknown model" errors at the provider.
+        model_id_override = body["model_id"].to_s.strip
+        model_id_override = nil if model_id_override.empty?
+
+        if model_id_override && !@agent_config.models.any? { |m| m["id"] == model_id_override }
+          return json_response(res, 400, { error: "Model not found in configuration" })
+        end
 
         # Create working directory if it doesn't exist
         # Allow multiple sessions in the same directory
         FileUtils.mkdir_p(working_dir)
 
-        session_id = build_session(name: name, working_dir: working_dir, profile: profile, source: source, model_override: model_override)
+        session_id = build_session(name: name, working_dir: working_dir, profile: profile, source: source, model_id: model_id_override)
         broadcast_session_update(session_id)
         json_response(res, 201, { session: @registry.session_summary(session_id) })
       end
@@ -2662,19 +2672,38 @@ module Clacky
       # @param working_dir [String] working directory for the agent
       # @param permission_mode [Symbol] :confirm_all (default, human present) or
       #   :auto_approve (unattended — suppresses request_user_feedback waits)
-      def build_session(name:, working_dir:, permission_mode: :confirm_all, profile: "general", source: :manual, model_override: nil)
+      def build_session(name:, working_dir:, permission_mode: :confirm_all, profile: "general", source: :manual, model_id: nil)
         session_id = Clacky::SessionManager.generate_id
         @registry.create(session_id: session_id)
 
-        client = @client_factory.call
         config = @agent_config.deep_copy
         config.permission_mode = permission_mode
-        
-        # Apply model override if provided
-        if model_override && config.current_model
-          config.current_model["model"] = model_override
-        end
-        
+
+        # Apply model override BEFORE creating the client — otherwise the
+        # client is built from the default model entry and may route through
+        # the wrong provider (e.g. sending a deepseek-v4-pro request to the
+        # Bedrock-format OpenClacky endpoint, which replies "unknown model").
+        #
+        # We use switch_model_by_id (not a name-based rewrite of
+        # current_model["model"]) because:
+        #   1. Ids uniquely identify an entry across providers; names can
+        #      collide between entries (deepseek vs dsk-deepseek aliases).
+        #   2. switch_model_by_id only flips per-session @current_model_id
+        #      in the dup'd config — it never mutates the shared @models
+        #      array (see AgentConfig#deep_copy's shared-ref contract).
+        #      A name rewrite would have leaked into every live session
+        #      AND corrupted the on-disk config at next save.
+        config.switch_model_by_id(model_id) if model_id
+
+        # Build client from the (possibly overridden) config so api format
+        # detection (Bedrock vs OpenAI vs Anthropic) uses the correct model.
+        client = Clacky::Client.new(
+          config.api_key,
+          base_url: config.base_url,
+          model: config.model_name,
+          anthropic_format: config.anthropic_format?
+        )
+
         broadcaster = method(:broadcast)
         ui = WebUIController.new(session_id, broadcaster)
         agent = Clacky::Agent.new(client, config, working_dir: working_dir, ui: ui, profile: profile,

data/lib/clacky/tools/terminal.rb

@@ -621,12 +621,39 @@ module Clacky
         log_file = SessionManager.allocate_log_file
         log_io   = File.open(log_file, "wb")
 
+        # Prevent the child process from inheriting the server's
+        # listening socket (port 7070) which would block hot_restart.
+        # PTY.spawn does not support close_others, so we temporarily
+        # set close_on_exec on the inherited fd — the kernel closes
+        # it in the child after exec while the parent keeps it open.
+        inherited_fd = ENV["CLACKY_INHERIT_FD"].to_i
+        if inherited_fd > 0
+          begin
+            inherited_io = IO.for_fd(inherited_fd)
+            inherited_io.autoclose = false
+            was_cloexec = inherited_io.close_on_exec?
+            inherited_io.close_on_exec = true
+          rescue StandardError
+            inherited_fd = 0
+          end
+        end
+
         reader, writer, pid = PTY.spawn(
           spawn_env, *args, chdir_args(cwd)
         )
         reader.sync = true
         writer.sync = true
 
+        # Restore original close_on_exec flag on the parent's fd so the
+        # server can continue accepting connections after hot_restart.
+        if inherited_fd > 0
+          begin
+            inherited_io.close_on_exec = was_cloexec
+          rescue StandardError
+            # best-effort
+          end
+        end
+
         begin
           writer.winsize = [40, 120]
         rescue StandardError

data/lib/clacky/version.rb

@@ -1,5 +1,5 @@
 # frozen_string_literal: true
 
 module Clacky
-  VERSION = "1.0.0.beta.1"
+  VERSION = "1.0.0.beta.2"
 end

data/lib/clacky/web/sessions.js

@@ -2153,7 +2153,13 @@ const Sessions = (() => {
 
       const agentProfile = agentSelect ? agentSelect.value : "general";
       const customName = nameInput ? nameInput.value.trim() : "";
-      const selectedModel = modelSelect ? modelSelect.value : "";
+      // The dropdown's value is the model's stable runtime id (see
+      // _populateModelDropdown). Using the id — not the model *name* — lets
+      // the backend switch to the right full model entry (api_key, base_url,
+      // anthropic_format) instead of mutating the current default entry's
+      // name in place, which caused "unknown model <name>" errors when the
+      // chosen model belonged to a different provider than the default.
+      const selectedModelId = modelSelect ? modelSelect.value : "";
       const workingDir = dirInput ? dirInput.value.trim() : "";
       const initProject = initCheckbox ? initCheckbox.checked : false;
 
@@ -2178,7 +2184,7 @@ const Sessions = (() => {
 
         // Add optional fields
         if (workingDir) payload.working_dir = workingDir;
-        if (selectedModel) payload.model = selectedModel;
+        if (selectedModelId) payload.model_id = selectedModelId;
 
         const res = await fetch("/api/sessions", {
           method: "POST",
@@ -2244,16 +2250,20 @@ const Sessions = (() => {
         return;
       }
 
-      // Add each configured model (CLI-style format)
+      // Add each configured model (CLI-style format).
+      // The option's value is the model's stable runtime id — not the bare
+      // model name — so the backend can switch to the exact model entry
+      // (with matching api_key / base_url / anthropic_format) when the user
+      // chooses a non-default model. See createFromModal + build_session.
       models.forEach(m => {
         const opt = document.createElement("option");
-        opt.value = m.model || "";
-        
+        opt.value = m.id || "";
+
         // Format: [default] abs-claude-sonnet-4-5 (clacky...8825)
         const typeBadge = m.type === "default" ? "[default] " : "";
         const label = `${typeBadge}${m.model} (${m.api_key_masked})`;
         opt.textContent = label;
-        
+
         // Pre-select default model
         if (m.type === "default") opt.selected = true;
         modelSelect.appendChild(opt);

data/scripts/install.ps1

@@ -57,6 +57,16 @@ function Test-IsAdmin {
         [Security.Principal.WindowsBuiltInRole]::Administrator)
 }
 
+function Get-SafeTempDir {
+    # Use [IO.Path]::GetTempPath() instead of $env:TEMP.
+    # $env:TEMP can return a short (8.3) path (e.g. C:\Users\USERNA~1\AppData\Local\Temp)
+    # on systems where the user profile path contains spaces or non-ASCII characters,
+    # which can break tools that don't handle 8.3 names correctly.
+    # [IO.Path]::GetTempPath() always returns the full long path.
+    $tempDir = [IO.Path]::GetTempPath().TrimEnd("\", "/")
+    return $tempDir
+}
+
 # Robust file download: try curl first (shows progress), fall back to
 # Invoke-WebRequest. Returns $true on success, $false on failure.
 function Invoke-Download {
@@ -79,7 +89,8 @@ function Invoke-Download {
 # Returns $true on match, or if the checksum file cannot be fetched (non-fatal).
 function Test-Sha256 {
     param([string]$FilePath, [string]$Sha256Url)
-    $sha256File = "$env:TEMP\download.sha256"
+    $safeTemp = Get-SafeTempDir
+    $sha256File = "$safeTemp\download.sha256"
     try {
         if (-not (Invoke-Download -Url $Sha256Url -OutFile $sha256File)) {
             Write-Warn "Could not download checksum file — skipping verification."
@@ -178,7 +189,8 @@ function Install-UbuntuRootfs {
         $sha256Url = $UBUNTU_WSL_AMD64_SHA256_URL
     }
 
-    $tarPath    = "$env:TEMP\ubuntu-wsl-$cpuArch.tar.gz"
+    $safeTemp   = Get-SafeTempDir
+    $tarPath    = "$safeTemp\ubuntu-wsl-$cpuArch.tar.gz"
     $installDir = $UBUNTU_WSL_DIR
 
     # Disk space check (~2 GB needed: 350 MB download + ~1.5 GB imported)
@@ -324,8 +336,10 @@ function Test-VirtualisationSupported {
     # Probe WSL2 with a minimal tar (512 zero bytes = valid tar EOF block)
     # Called only after WSL feature is confirmed enabled (Main already checked).
     Write-Info "Probing WSL2 availability..."
-    $probeTar = "$env:TEMP\wsl_probe.tar"
-    $probeDir = "$env:TEMP\wsl_probe"
+
+    $safeTemp = Get-SafeTempDir
+    $probeTar = "$safeTemp\wsl_probe.tar"
+    $probeDir = "$safeTemp\wsl_probe"
     $ok = $false
     try {
         $bytes = New-Object byte[] 512
@@ -369,7 +383,8 @@ function Install-WslKernel {
         $url = $WSL_UPDATE_URL_X64
     }
 
-    $msiPath = "$env:TEMP\wsl_update.msi"
+    $safeTemp = Get-SafeTempDir
+    $msiPath = "$safeTemp\wsl_update.msi"
     Write-Step "Downloading WSL kernel update ($cpuArch)..."
     if (-not (Invoke-Download -Url $url -OutFile $msiPath)) {
         Write-Fail "Failed to download WSL kernel update. Check your network and try again."

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: openclacky
 version: !ruby/object:Gem::Version
-  version: 1.0.0.beta.1
+  version: 1.0.0.beta.2
 platform: ruby
 authors:
 - windy