fix: run paperclip from cloned repo via pnpm dev:once - #1352
Conversation
The global bun install flattens pino@10 + pino-http@10.5 together, but pino-http needs pino@9. The repo lockfile resolves this correctly with nested dependencies. Running from the repo avoids the crash.
|
You do not have enough credits to review this pull request. Please purchase more credits to continue. |
|
Caution Review failedThe pull request is closed. ℹ️ Recent review info⚙️ Run configurationConfiguration used: Organization UI Review profile: CHILL Plan: Pro Run ID: ⛔ Files ignored due to path filters (1)
📒 Files selected for processing (2)
Disabled knowledge base sources:
📝 WalkthroughSummary by CodeRabbit
WalkthroughSystemd user service for Paperclip now runs the repository-local binary at Changes
Sequence Diagram(s)sequenceDiagram
autonumber
participant Systemd as Systemd.user (paperclip)
participant Env as EnvFiles (dotfiles/.env, .paperclip/.../.env)
participant FS as Filesystem (dotfiles/node_modules)
participant Binary as LocalPaperclip (paperclipai)
rect rgba(200,200,255,0.5)
Systemd->>Env: source EnvironmentFile(s)
Env-->>Systemd: exported env vars (HOST, MODE, ALLOWED_HOSTNAMES)
end
rect rgba(200,255,200,0.5)
Systemd->>FS: set WorkingDirectory / locate `dotfiles/node_modules/.bin/paperclipai`
FS-->>Systemd: binary path resolved
end
rect rgba(255,200,200,0.5)
Systemd->>Binary: ExecStart with env (PATH, HOME, sourced envs)
Binary-->>Systemd: process started (stdout/stderr -> journal)
end
Estimated code review effort🎯 3 (Moderate) | ⏱️ ~20 minutes Possibly related PRs
Poem
🚥 Pre-merge checks | ✅ 1 | ❌ 2❌ Failed checks (1 warning, 1 inconclusive)
✅ Passed checks (1 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing Touches🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
Mesa DescriptionTL;DRRun Paperclip from the cloned repository via What changed?
Description generated by Mesa. Update settings |
![gemini-code-assist[bot]](https://avatars.githubusercontent.com/in/956858?s=60&v=4){kind=small}
There was a problem hiding this comment.
Code Review
This pull request updates the Paperclip systemd service to run from a cloned repository using pnpm dev:once instead of a global bun installation, addressing dependency version mismatches. It also adds several environment variables and configures EnvironmentFile paths. A review comment suggests prefixing these environment file paths with a hyphen to prevent the service from failing if the files are missing.
| EnvironmentFile = [ | ||
| "${homeDir}/dotfiles/.env" | ||
| "${homeDir}/.paperclip/instances/default/.env" | ||
| ]; |
There was a problem hiding this comment.
The EnvironmentFile directive in systemd will cause the service to fail to start if any of the specified files do not exist. Since these files (such as ~/dotfiles/.env or the instance-specific .env) might be missing on a fresh installation or before manual configuration, it is more robust to prefix the paths with -. This tells systemd to ignore the file if it is missing, allowing the service to attempt to start and let the application handle missing environment variables (consistent with the logic in hydrate.sh).
EnvironmentFile = [
"-${homeDir}/dotfiles/.env"
"-${homeDir}/.paperclip/instances/default/.env"
];
![coderabbitai[bot]](https://avatars.githubusercontent.com/in/347564?s=60&v=4){kind=small}
There was a problem hiding this comment.
Actionable comments posted: 1
🧹 Nitpick comments (2)
home-manager/modules/paperclip/default.nix (2)
10-10: Repository clone is assumed but not validated.The service depends on the repo being cloned at
~/ghq/github.com/paperclipai/paperclipwith dependencies installed. Consider adding an assertion or extending thepaperclipSetupactivation to verify the repo exists, or document this as a manual prerequisite.Also applies to: 47-47
🤖 Prompt for AI Agents
Verify each finding against the current code and only fix it if needed. In `@home-manager/modules/paperclip/default.nix` at line 10, Add an explicit existence check for the repository path referenced by repoDir (repoDir = "${homeDir}/ghq/github.com/paperclipai/paperclip") and fail fast or log a clear error if it is missing; update the paperclipSetup activation (paperclipSetup) to assert the directory exists (or run a clone step) and verify dependencies are installed, or alternatively add a clear README/activation note documenting the manual prerequisite that the repo must be cloned to that path with dependencies installed. Ensure the check uses the same repoDir symbol so it stays correct if the path changes.
40-40: Hardcoded Docker bridge IP may not be portable.
172.17.0.1is Docker's default bridge gateway, but this can change if Docker is configured with custom networks or ifdocker0bridge settings are modified. Consider deriving this dynamically or documenting the assumption.🤖 Prompt for AI Agents
Verify each finding against the current code and only fix it if needed. In `@home-manager/modules/paperclip/default.nix` at line 40, The allowed-hostnames string currently hardcodes the Docker bridge IP ("PAPERCLIP_ALLOWED_HOSTNAMES=paperclip.shunkakinoki.com,172.17.0.1"), which is not portable; update the configuration so the second entry is derived or configurable: replace the hardcoded 172.17.0.1 with a computed value (e.g., resolve the docker0 gateway at activation/runtime or use a nixOS module option/environment variable that can be set per-host) or explicitly document the assumption and expose an option like paperclip.allowedHostnames to append the host IP; ensure you modify the literal "PAPERCLIP_ALLOWED_HOSTNAMES=paperclip.shunkakinoki.com,172.17.0.1" to use the chosen dynamic/configurable value.
🤖 Prompt for all review comments with AI agents
Verify each finding against the current code and only fix it if needed.
Inline comments:
In `@home-manager/modules/paperclip/default.nix`:
- Around line 43-46: The EnvironmentFile entries under EnvironmentFile will
cause service failure if the files are missing; update the EnvironmentFile array
to mark the entries optional by prefixing each path with "-" (e.g.
"-${homeDir}/dotfiles/.env") so systemd won't fail when a file is absent, or
alternatively add an assertion or an activation script that creates/validates
the files before the service starts; locate the EnvironmentFile attribute in the
Nix service declaration and apply one of these fixes to ensure the service
doesn't fail on missing files.
---
Nitpick comments:
In `@home-manager/modules/paperclip/default.nix`:
- Line 10: Add an explicit existence check for the repository path referenced by
repoDir (repoDir = "${homeDir}/ghq/github.com/paperclipai/paperclip") and fail
fast or log a clear error if it is missing; update the paperclipSetup activation
(paperclipSetup) to assert the directory exists (or run a clone step) and verify
dependencies are installed, or alternatively add a clear README/activation note
documenting the manual prerequisite that the repo must be cloned to that path
with dependencies installed. Ensure the check uses the same repoDir symbol so it
stays correct if the path changes.
- Line 40: The allowed-hostnames string currently hardcodes the Docker bridge IP
("PAPERCLIP_ALLOWED_HOSTNAMES=paperclip.shunkakinoki.com,172.17.0.1"), which is
not portable; update the configuration so the second entry is derived or
configurable: replace the hardcoded 172.17.0.1 with a computed value (e.g.,
resolve the docker0 gateway at activation/runtime or use a nixOS module
option/environment variable that can be set per-host) or explicitly document the
assumption and expose an option like paperclip.allowedHostnames to append the
host IP; ensure you modify the literal
"PAPERCLIP_ALLOWED_HOSTNAMES=paperclip.shunkakinoki.com,172.17.0.1" to use the
chosen dynamic/configurable value.
🪄 Autofix (Beta)
Fix all unresolved CodeRabbit comments on this PR:
- Push a commit to this branch (recommended)
- Create a new PR with the fixes
ℹ️ Review info
⚙️ Run configuration
Configuration used: Organization UI
Review profile: CHILL
Plan: Pro
Run ID: ab480ad7-89ab-4c62-8bd5-1fee82c61d4f
📒 Files selected for processing (1)
home-manager/modules/paperclip/default.nix
| EnvironmentFile = [ | ||
| "${homeDir}/dotfiles/.env" | ||
| "${homeDir}/.paperclip/instances/default/.env" | ||
| ]; |
There was a problem hiding this comment.
EnvironmentFile entries will cause service failure if files are missing.
Systemd's EnvironmentFile directive will fail service startup if the specified file doesn't exist. Consider prefixing with - to make them optional if that's acceptable, or ensure the files are created before service start.
EnvironmentFile = [
- "${homeDir}/dotfiles/.env"
- "${homeDir}/.paperclip/instances/default/.env"
+ "-${homeDir}/dotfiles/.env"
+ "-${homeDir}/.paperclip/instances/default/.env"
];Alternatively, if these files are strictly required, add assertions or activation scripts to validate their existence.
📝 Committable suggestion
‼️ IMPORTANT
Carefully review the code before committing. Ensure that it accurately replaces the highlighted code, contains no missing lines, and has no issues with indentation. Thoroughly test & benchmark the code to ensure it meets the requirements.
| EnvironmentFile = [ | |
| "${homeDir}/dotfiles/.env" | |
| "${homeDir}/.paperclip/instances/default/.env" | |
| ]; | |
| EnvironmentFile = [ | |
| "-${homeDir}/dotfiles/.env" | |
| "-${homeDir}/.paperclip/instances/default/.env" | |
| ]; |
🤖 Prompt for AI Agents
Verify each finding against the current code and only fix it if needed.
In `@home-manager/modules/paperclip/default.nix` around lines 43 - 46, The
EnvironmentFile entries under EnvironmentFile will cause service failure if the
files are missing; update the EnvironmentFile array to mark the entries optional
by prefixing each path with "-" (e.g. "-${homeDir}/dotfiles/.env") so systemd
won't fail when a file is absent, or alternatively add an assertion or an
activation script that creates/validates the files before the service starts;
locate the EnvironmentFile attribute in the Nix service declaration and apply
one of these fixes to ensure the service doesn't fail on missing files.
There was a problem hiding this comment.
Pull request overview
Updates the Home Manager systemd user service for Paperclip to run from a cloned repository via pnpm dev:once, avoiding dependency resolution issues seen with a global bun install.
Changes:
- Switch
ExecStartto~/.bun/bin/pnpm dev:onceand setWorkingDirectoryto the cloned Paperclip repo path. - Add
EnvironmentFileentries to load shared (~/dotfiles/.env) and instance (~/.paperclip/instances/default/.env) environment variables. - Set additional runtime env vars (
HOST,PAPERCLIP_DEPLOYMENT_MODE,PAPERCLIP_ALLOWED_HOSTNAMES).
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
| "${homeDir}/dotfiles/.env" | ||
| "${homeDir}/.paperclip/instances/default/.env" |
There was a problem hiding this comment.
EnvironmentFile entries are required by systemd; if either ${homeDir}/dotfiles/.env or ${homeDir}/.paperclip/instances/default/.env is missing, the unit will fail to start. Either ensure these files (and the instances/default directory) are created with appropriate permissions during activation, or prefix the paths with - to make them optional and handle missing values another way.
| "${homeDir}/dotfiles/.env" | |
| "${homeDir}/.paperclip/instances/default/.env" | |
| "-${homeDir}/dotfiles/.env" | |
| "-${homeDir}/.paperclip/instances/default/.env" |
| "${homeDir}/dotfiles/.env" | ||
| "${homeDir}/.paperclip/instances/default/.env" | ||
| ]; | ||
| WorkingDirectory = "${repoDir}"; |
There was a problem hiding this comment.
The service now WorkingDirectorys into ${repoDir}, but this module doesn't ensure the repo exists. If the directory is missing (fresh machine, different ghq root, etc.), systemd will fail with a CHDIR error and keep restarting. Consider adding a ConditionPathIsDirectory= / AssertPathIsDirectory= in the unit, or an ExecStartPre check that exits with a clear log message when the repo isn't present.
![cubic-dev-ai[bot]](https://avatars.githubusercontent.com/in/1082092?s=60&v=4){kind=small}
There was a problem hiding this comment.
1 issue found across 1 file
Prompt for AI agents (unresolved issues)
Check if these issues are valid — if so, understand the root cause of each and fix them. If appropriate, use sub-agents to investigate and fix each issue separately.
<file name="home-manager/modules/paperclip/default.nix">
<violation number="1" location="home-manager/modules/paperclip/default.nix:44">
P1: Systemd's `EnvironmentFile` directive treats unprefixed paths as required — the service will refuse to start if either file is missing (e.g. fresh machine, before manual setup). Prefix with `-` to make them optional:
```nix
EnvironmentFile = [
"-${homeDir}/dotfiles/.env"
"-${homeDir}/.paperclip/instances/default/.env"
];
```</violation>
</file>
Reply with feedback, questions, or to request a fix. Tag @cubic-dev-ai to re-run a review.
| "${homeDir}/dotfiles/.env" | ||
| "${homeDir}/.paperclip/instances/default/.env" |
There was a problem hiding this comment.
P1: Systemd's EnvironmentFile directive treats unprefixed paths as required — the service will refuse to start if either file is missing (e.g. fresh machine, before manual setup). Prefix with - to make them optional:
EnvironmentFile = [
"-${homeDir}/dotfiles/.env"
"-${homeDir}/.paperclip/instances/default/.env"
];Prompt for AI agents
Check if this issue is valid — if so, understand the root cause and fix it. At home-manager/modules/paperclip/default.nix, line 44:
<comment>Systemd's `EnvironmentFile` directive treats unprefixed paths as required — the service will refuse to start if either file is missing (e.g. fresh machine, before manual setup). Prefix with `-` to make them optional:
```nix
EnvironmentFile = [
"-${homeDir}/dotfiles/.env"
"-${homeDir}/.paperclip/instances/default/.env"
];
```</comment>
<file context>
@@ -27,14 +30,21 @@ lib.mkIf host.isKyber {
];
- WorkingDirectory = "${homeDir}/.paperclip";
+ EnvironmentFile = [
+ "${homeDir}/dotfiles/.env"
+ "${homeDir}/.paperclip/instances/default/.env"
+ ];
</file context>
| "${homeDir}/dotfiles/.env" | |
| "${homeDir}/.paperclip/instances/default/.env" | |
| "-${homeDir}/dotfiles/.env" | |
| "-${homeDir}/.paperclip/instances/default/.env" |
…ules The bun flat hoisting was resolving pino@10 which is incompatible with pino-http@10.5. Pinning pino to 9.14.0 via overrides matches the paperclip repo's lockfile resolution and fixes the crash.
There was a problem hiding this comment.
Actionable comments posted: 1
♻️ Duplicate comments (1)
home-manager/modules/paperclip/default.nix (1)
43-46:⚠️ Potential issue | 🟠 Major
EnvironmentFileentries can still fail startup when files are absent.Line 43-46 should be optional (
-prefix) unless you strictly guarantee file creation before this unit starts. The second file is especially risky given current hydration behavior.Suggested fix
EnvironmentFile = [ - "${homeDir}/dotfiles/.env" - "${homeDir}/.paperclip/instances/default/.env" + "-${homeDir}/dotfiles/.env" + "-${homeDir}/.paperclip/instances/default/.env" ];#!/bin/bash set -euo pipefail echo "EnvironmentFile references:" rg -n --type nix 'EnvironmentFile|\\.paperclip/instances/default/\\.env|dotfiles/\\.env' echo echo "Search for creation of the default instance .env:" rg -n --type sh --type nix 'instances/default/.+\\.env|touch.+\\.env|cat.+>.+\\.env|writeText.*\\.env'🤖 Prompt for AI Agents
Verify each finding against the current code and only fix it if needed. In `@home-manager/modules/paperclip/default.nix` around lines 43 - 46, The EnvironmentFile entries in the systemd unit (EnvironmentFile = [...]) can cause unit startup failures if the files don't exist; change the two entries referencing "${homeDir}/dotfiles/.env" and "${homeDir}/.paperclip/instances/default/.env" to optional by prefixing each path with a "-" so systemd ignores missing files, or alternatively ensure creation of the default instance .env before unit activation (e.g., via a setup/hydration step that writes "${homeDir}/.paperclip/instances/default/.env"); update the EnvironmentFile array in default.nix accordingly and keep the variable name homeDir and the exact path strings to locate the change.
🧹 Nitpick comments (1)
home-manager/modules/paperclip/default.nix (1)
38-40: Consider deriving runtime env values from one source to avoid drift.These values are now hardcoded here while related host/deployment values are also templated in Paperclip hydration config. Centralizing them (Nix attr/let binding reused by both) will reduce config skew.
🤖 Prompt for AI Agents
Verify each finding against the current code and only fix it if needed. In `@home-manager/modules/paperclip/default.nix` around lines 38 - 40, The three hardcoded runtime env values "HOST", "PAPERCLIP_DEPLOYMENT_MODE", and "PAPERCLIP_ALLOWED_HOSTNAMES" in home-manager/modules/paperclip/default.nix should be derived from a single Nix attribute/let binding (e.g., paperclip.host, paperclip.deploymentMode, paperclip.allowedHostnames) and referenced wherever Paperclip hydration templating is generated; update default.nix to introduce those attributes in the module options/let and replace the literal strings with references to those attributes so both the runtime env block and the hydration config/template reuse the same values to avoid drift.
🤖 Prompt for all review comments with AI agents
Verify each finding against the current code and only fix it if needed.
Inline comments:
In `@home-manager/modules/paperclip/default.nix`:
- Line 10: The systemd unit can hard-fail if the repo directory (repoDir =
"${homeDir}/ghq/github.com/paperclipai/paperclip") is missing because
paperclipSetup only creates directories; add a ConditionPathExists check to the
generated unit so systemd will skip/explicitly fail when the repo is absent.
Modify the code that emits the unit (look for the unit section referencing
WorkingDirectory and ExecStart in the paperclipSetup/unit or service generation
logic) to include ConditionPathExists=<path> using the same repoDir symbol,
ensuring the path guard is added alongside the existing WorkingDirectory and
ExecStart entries.
---
Duplicate comments:
In `@home-manager/modules/paperclip/default.nix`:
- Around line 43-46: The EnvironmentFile entries in the systemd unit
(EnvironmentFile = [...]) can cause unit startup failures if the files don't
exist; change the two entries referencing "${homeDir}/dotfiles/.env" and
"${homeDir}/.paperclip/instances/default/.env" to optional by prefixing each
path with a "-" so systemd ignores missing files, or alternatively ensure
creation of the default instance .env before unit activation (e.g., via a
setup/hydration step that writes
"${homeDir}/.paperclip/instances/default/.env"); update the EnvironmentFile
array in default.nix accordingly and keep the variable name homeDir and the
exact path strings to locate the change.
---
Nitpick comments:
In `@home-manager/modules/paperclip/default.nix`:
- Around line 38-40: The three hardcoded runtime env values "HOST",
"PAPERCLIP_DEPLOYMENT_MODE", and "PAPERCLIP_ALLOWED_HOSTNAMES" in
home-manager/modules/paperclip/default.nix should be derived from a single Nix
attribute/let binding (e.g., paperclip.host, paperclip.deploymentMode,
paperclip.allowedHostnames) and referenced wherever Paperclip hydration
templating is generated; update default.nix to introduce those attributes in the
module options/let and replace the literal strings with references to those
attributes so both the runtime env block and the hydration config/template reuse
the same values to avoid drift.
🪄 Autofix (Beta)
Fix all unresolved CodeRabbit comments on this PR:
- Push a commit to this branch (recommended)
- Create a new PR with the fixes
ℹ️ Review info
⚙️ Run configuration
Configuration used: Organization UI
Review profile: CHILL
Plan: Pro
Run ID: a60b45b7-0b1a-4b70-804d-c3c68337c83b
📒 Files selected for processing (1)
home-manager/modules/paperclip/default.nix
| let | ||
| inherit (inputs) host; | ||
| homeDir = config.home.homeDirectory; | ||
| repoDir = "${homeDir}/ghq/github.com/paperclipai/paperclip"; |
There was a problem hiding this comment.
🧩 Analysis chain
🏁 Script executed:
#!/bin/bash
set -euo pipefail
# Verify whether repo provisioning exists anywhere in Home Manager/Nix code.
rg -n --type nix --type sh 'paperclipai/paperclip|ghq/github.com/paperclipai/paperclip|ConditionPathExists'Repository: shunkakinoki/dotfiles
Length of output: 173
🏁 Script executed:
#!/bin/bash
# View the full paperclip service configuration
cat -n home-manager/modules/paperclip/default.nixRepository: shunkakinoki/dotfiles
Length of output: 2337
🏁 Script executed:
#!/bin/bash
# Search for ghq initialization, git clone, or Home Manager activation scripts
rg -n 'ghq|git clone|activation' --type nix --type sh -A 3 -B 3Repository: shunkakinoki/dotfiles
Length of output: 37004
Add a guard for missing repo directory before service start.
The paperclipSetup activation script (lines 15-18) only creates directories—it does not clone the repository. If ${repoDir} doesn't exist at service start, WorkingDirectory (line 47) and ExecStart (line 33) will cause systemd to hard-fail the unit. Add ConditionPathExists to make the failure explicit rather than flaky.
Suggested hardening
systemd.user.services.paperclip = {
Unit = {
Description = "Paperclip AI agent orchestration platform";
After = [ "network-online.target" ];
Wants = [ "network-online.target" ];
+ ConditionPathExists = "${repoDir}/server/src/index.ts";
};🤖 Prompt for AI Agents
Verify each finding against the current code and only fix it if needed.
In `@home-manager/modules/paperclip/default.nix` at line 10, The systemd unit can
hard-fail if the repo directory (repoDir =
"${homeDir}/ghq/github.com/paperclipai/paperclip") is missing because
paperclipSetup only creates directories; add a ConditionPathExists check to the
generated unit so systemd will skip/explicitly fail when the repo is absent.
Modify the code that emits the unit (look for the unit section referencing
WorkingDirectory and ExecStart in the paperclipSetup/unit or service generation
logic) to include ConditionPathExists=<path> using the same repoDir symbol,
ensuring the path guard is added alongside the existing WorkingDirectory and
ExecStart entries.
* feat: add paperclip service (#1342) * feat: add paperclip service - Systemd service on kyber: runs `paperclipai run` via bun, depends on docker-postgres - Config via builtins.toJSON: external postgres on kyber, embedded on macOS - Setup script creates paperclip database on docker-postgres - Makefile target: `make systemctl-paperclip` * fix: add shellcheck disable and shell test coverage for paperclip * fix: correct shellcheck disable directive syntax * refactor: use config.template.json pattern for paperclip * refactor: rename setup.sh to hydrate.sh for paperclip * test: add auto-switch hook tests and update coverage spec * fix: remove EnvironmentFile from paperclip service (#1344) * fix: paperclip authenticated mode and required config fields (#1345) * fix: remove EnvironmentFile from paperclip service * fix: use authenticated mode on kyber, add required meta/logging fields * fix: add allowedHostnames for paperclip.shunkakinoki.com * fix: format config template json * fix: use trust auth for docker-postgres (#1346) POSTGRES_HOST_AUTH_METHOD=trust removes password auth for all connections. The password kept going out of sync after crash recovery, causing paperclip and other services to fail with 'password authentication failed'. Safe since postgres is only reachable from the host. * fix: use k8s postgres via DATABASE_URL, remove docker-postgres dep, add authenticated mode (#1347) * fix: use bun runtime for paperclip (pino-http node crash), k8s postgres via DATABASE_URL (#1348) * fix: use nix-profile bun path for paperclip service (#1350) * fix: add caret prefix to paperclipai dependency (#1349) * fix: use extra-substituters to avoid untrusted user warnings (#1351) * fix: run paperclip from cloned repo via pnpm dev:once (#1352) * fix: run paperclip from cloned repo via pnpm dev:once The global bun install flattens pino@10 + pino-http@10.5 together, but pino-http needs pino@9. The repo lockfile resolves this correctly with nested dependencies. Running from the repo avoids the crash. * fix: use bun run server/src/index.ts instead of pnpm dev:once * fix: pin pino@9.14.0 override, run paperclipai from dotfiles node_modules The bun flat hoisting was resolving pino@10 which is incompatible with pino-http@10.5. Pinning pino to 9.14.0 via overrides matches the paperclip repo's lockfile resolution and fixes the crash. * fix: use global bun paperclipai with pino override (#1353) * fix: use global ~/.bun/bin/paperclipai with pino override Propagate overrides from dotfiles package.json to ~/.bun/install/global/ so the global binary resolves pino@9.14.0 correctly. * test: add tests for npm-globals dependency overrides * fix: resolve GitHub Actions failures and code review issues - Fix non-portable \s regex to [[:space:]] in auto-switch.sh (shfmt compat) - Add jq dependency check alongside cswap - Use printf instead of echo for safer output - Fix claude-swap version from >=1.1.5 (non-existent) to >=0.7.1 - Add auto-switch.sh to Nix deployment config (default.nix) - Sort covered_scripts list alphabetically in coverage_spec.sh https://claude.ai/code/session_012GyQBesQGF1asTfKebWyLM --------- Co-authored-by: Claude <noreply@anthropic.com>
Summary
Run paperclip from
~/ghq/github.com/paperclipai/paperclipusingpnpm dev:onceinstead of the global bun install.Root cause
The global
bun install -g paperclipaiflattenspino@10andpino-http@10.5together, butpino-http@10.5requirespino@9. The repo's lockfile correctly nestspino@9.14.0insidepino-http, avoiding the crash.Changes
ExecStart:pnpm dev:once(runs from the cloned repo)WorkingDirectory:~/ghq/github.com/paperclipai/paperclipEnvironmentFile: loads both~/dotfiles/.env(DATABASE_URL) and instance.env(secrets)HOST=0.0.0.0,PAPERCLIP_DEPLOYMENT_MODE=authenticated,PAPERCLIP_ALLOWED_HOSTNAMESTested
10/10 sequential requests returned 200 with
pnpm dev:once— no crashes.Summary by cubic
Run Paperclip using the local
paperclipaibinary and pinpino@9.14.0to fix crashes from apino/pino-httpversion mismatch. The systemd service now uses the local binary, loads env files, and sets required env vars.Bug Fixes
${HOME}/dotfiles/node_modules/.bin/paperclipai run --no-repair;WorkingDirectory=${HOME}/.paperclip.~/dotfiles/.envand the instance.env; setsHOST=0.0.0.0,PAPERCLIP_DEPLOYMENT_MODE=authenticated,PAPERCLIP_ALLOWED_HOSTNAMES.Dependencies
pinoto9.14.0(removepino-httpoverride); lock now resolvespino-http@10.5.0.bun.lockupdated accordingly (introduces@pinojs/redact, replaces oldfast-redact).Written for commit 33995be. Summary will update on new commits.