Repository navigation
fix(crowdgpt): runner files inside the crate (dev2 deploy) + build on Windows - #187
Merged
Merged
Conversation
…ilds compile The c0mpute.com deploy builds from a context with only node/ copied, so the include_str! of plugins/crowdgpt/runner broke every dev2 deploy since #181. The Dockerfile and watchdog now live in node/crates/c0mpute-core/crowdgpt-runner; the image workflow builds from there. No behavior change. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
ThreatCrush Security Scan13 finding(s) HIGH/CRITICAL: 1 | MEDIUM: 8 | LOW: 4
Snippets are redacted; ThreatCrush never prints matched credential material. |
v0.2.89's windows-x86_64 release build failed on std::os::unix in crowdgpt.rs. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
ralyodio
changed the title
Collaborator
Author
|
gitleaks fails on commit 7fb45f2 (.github/workflows/release.yml) which exists only on the unrelated branch fix/release-signing; the history scan picks up every fetched ref. This PR does not touch that file. Merging on the other green checks (build + test, image build, semgrep, CodeQL). |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Two fixes for #181:
node/, andcrowdgpt.rsdidinclude_str!ofplugins/crowdgpt/runner/*. The Dockerfile and watchdog move tonode/crates/c0mpute-core/crowdgpt-runner/; the image workflow builds from there; the Dockerfile.web COPY from feat(crowdgpt): donate idle compute to CrowdGPT (opt-in, preempted by paid work) #181 is dropped.std::os::unix(uid/gid of the cache dir, 0600 token file). Those are nowcfg(unix); on other platforms the container runs as its image's non-root user.No behavior change on Linux.
🤖 Generated with Claude Code