happyskills 0.47.0 → 0.47.1

package/CHANGELOG.md

@@ -7,6 +7,15 @@ and this project adheres to [Semantic Versioning](https://semver.org/).
 
 ## [Unreleased]
 
+## [0.47.1] - 2026-05-22
+
+### Fixed
+- Fix `search --with-rerank` silently dropping every rerank field from the API response. The HTTP client's default `{ data: ... }` envelope unwrap was discarding `mode`, `match_notice`, `workspace_match`, and (when opted-in) `rerank_digests`, `rerank_system_prompt`, `rerank_prompt_version`, `rerank_response_schema` from `POST /repos:search`. The CLI's envelope emission saw `digests.length === 0` and fell through to "no protocol envelope" → the agent rendered the baseline ranking instead of running the LLM rerank. With this fix, `dispatch_search()` opts out of the auto-unwrap via a new `unwrap: false` option on `client.request()` and the caller now receives the full envelope. Verified end-to-end against production: natural-language query returns `next_step.action == "rank_digests_inline"` with 50 digests + the 1519-char system prompt + the strict json_schema; slug-shape query still emits `next_step: null` (no false-positive protocol fire). The bug had been latent since `0.47.0`'s introduction of `--with-rerank` because the existing rendering code in `search.js` already had a defensive fallback for both response shapes (`Array.isArray(response) ? response : response?.data`), but the rerank path requires the top-level fields specifically.
+- Plain `search --json` (without `--with-rerank`) now correctly carries `data.mode` (e.g. `'semantic'` / `'fuzzy_slug'` / `'fuzzy_scoped'`) in the JSON output. Previously `mode` was always `null` because the client auto-unwrap silently dropped it. The human-readable output also gains a `[mode]` annotation next to the query header, and the yellow `match_notice` warning ("No strong matches found…") now shows up on weak-result searches — both were already coded but never received their inputs.
+
+### Changed
+- `client.request()` (`cli/src/api/client.js`) gains an `unwrap` option (default `true`, matching the existing behavior). Set `unwrap: false` to receive the raw response body instead of the auto-unwrapped `data.data`. Required for endpoints (currently only `POST /repos:search`) that emit metadata fields at the top level of the response alongside the `data` payload. Existing callers are unaffected because the default preserves current behavior.
+
 ## [0.47.0] - 2026-05-21
 
 ### Added

package/package.json

@@ -1,6 +1,6 @@
 {
 	"name": "happyskills",
-	"version": "0.47.0",
+	"version": "0.47.1",
 	"description": "Package manager for AI agent skills",
 	"license": "SEE LICENSE IN LICENSE",
 	"author": "Nicolas Dao <nic@cloudlesslabs.com> (https://cloudlesslabs.com)",

package/src/api/client.js

@@ -7,7 +7,7 @@ const { load_token } = require('../auth/token_store')
 const get_base_url = () => process.env.HAPPYSKILLS_API_URL || API_URL
 
 const request = (method, path, options = {}) => catch_errors(`API ${method} ${path} failed`, async () => {
-	const { body, auth = true, raw_response = false, headers: extra_headers = {} } = options
+	const { body, auth = true, raw_response = false, unwrap = true, headers: extra_headers = {} } = options
 	const url = `${get_base_url()}${path}`
 	const headers = { ...extra_headers }
 
@@ -64,6 +64,11 @@ const request = (method, path, options = {}) => catch_errors(`API ${method} ${pa
 		throw new ApiError(err_msg, res.status, err_code)
 	}
 
+	// Auto-unwrap `{ data: ... }` envelope by default. Callers that need the
+	// full response (e.g. endpoints that emit top-level metadata fields
+	// alongside `data` — see `dispatch_search` for `/repos:search`'s
+	// `mode` / `rerank_digests` / `match_notice` fields) pass `unwrap: false`.
+	if (!unwrap) return data
 	return data?.data !== undefined ? data.data : data
 })
 

package/src/api/repos.js

@@ -149,7 +149,12 @@ const dispatch_search = (query, options = {}) => catch_errors('Search failed', a
 	// protocol. Server returns digests + system prompt + json_schema only
 	// when this is true AND the dispatcher routes to mode='semantic'.
 	if (options.with_rerank_digests) body.with_rerank_digests = true
-	const [errors, data] = await client.post('/repos:search', body, { auth: true })
+	// `/repos:search` returns top-level metadata alongside `data` —
+	// `mode`, `match_notice`, `workspace_match`, and (when opted-in)
+	// `rerank_digests`, `rerank_system_prompt`, `rerank_prompt_version`,
+	// `rerank_response_schema`. The client's default unwrap would drop these
+	// fields entirely. Opt out so the caller sees the full envelope.
+	const [errors, data] = await client.post('/repos:search', body, { auth: true, unwrap: false })
 	if (errors) throw errors[errors.length - 1]
 	return data
 })