git: stop tracking bundled build artifacts#2041
Closed
0oAstro wants to merge 7968 commits intoopenclaw:mainfrom
Closed
git: stop tracking bundled build artifacts#20410oAstro wants to merge 7968 commits intoopenclaw:mainfrom
0oAstro wants to merge 7968 commits intoopenclaw:mainfrom
Conversation
- Increase recommended memory to 2GB (512MB/1GB OOM) - Add OOM symptoms (SIGABRT, v8 allocation errors) - Fix lock file path (/data/gateway.*.lock) - Add complete config example with failover, auth, bindings - Document Discord token from env var vs config - Add machine update commands for command/memory changes - Add config writing tips (echo+tee, sftp caveats) Learned from FLAWD deployment debugging.
) Co-authored-by: Glucksberg <[email protected]>
…penclaw#1576) * fix(agents): treat request-aborted as timeout for fallback * test(e2e): add provider timeout fallback
…ements When native slash commands are executed in Telegram topics/forums, the originating topic context was not being preserved. This caused sub-agent announcements to be delivered to the wrong topic. Root cause: Native slash command context did not set OriginatingChannel and OriginatingTo, causing session delivery context to fallback to the user's personal ID instead of the group ID + topic. Fix: Added OriginatingChannel and OriginatingTo to native slash command context, ensuring topic information is preserved for sub-agent announcements. Related session fields: - lastThreadId: preserved via MessageThreadId - lastTo: now correctly set to group ID via OriginatingTo - deliveryContext: includes threadId for proper routing
Thanks @tyler6204 Co-authored-by: Tyler Yust <[email protected]>
Co-authored-by: Aaron Ng <[email protected]>
…oxy (openclaw#1795) * fix(gateway): prevent auth bypass when behind unconfigured reverse proxy When proxy headers (X-Forwarded-For, X-Real-IP) are present but gateway.trustedProxies is not configured, the gateway now treats connections as non-local. This prevents a scenario where all proxied requests appear to come from localhost and receive automatic trust. Previously, running behind nginx/Caddy without configuring trustedProxies would cause isLocalClient=true for all external connections, potentially bypassing authentication and auto-approving device pairing. The gateway now logs a warning when this condition is detected, guiding operators to configure trustedProxies for proper client IP detection. Also adds documentation for reverse proxy security configuration. * fix: harden reverse proxy auth (openclaw#1795) (thanks @orlyjamie) --------- Co-authored-by: orlyjamie <[email protected]> Co-authored-by: Peter Steinberger <[email protected]>
- Add resumeArgs to DEFAULT_CLAUDE_BACKEND for proper --resume flag usage - Fix gateway not preserving cliSessionIds/claudeCliSessionId in nextEntry - Add test for CLI session ID preservation in gateway agent handler - Update docs with new resumeArgs default
CLI backends (claude-cli etc) don't emit streaming assistant events, causing TUI to show "(no output)" despite correct processing. Now emits assistant event with final text before lifecycle end so server-chat buffer gets populated for WebSocket clients.
These files are generated at build time and shouldn't be committed: - dist/control-ui assets (JS/CSS bundles) - src/canvas-host/a2ui bundle files This removes ~100MB+ of bloat from git history by no longer tracking repeatedly regenerated bundle files. Add to .gitignore to prevent accidental re-addition. Co-Authored-By: Claude <[email protected]>
Contributor
There was a problem hiding this comment.
Pull request overview
This PR removes generated bundle files from git tracking to reduce repository bloat. The changes remove approximately 21,000 lines of JavaScript/CSS bundle files and add .gitignore entries to prevent them from being re-committed.
Changes:
- Removes
dist/control-ui/build artifacts (HTML, CSS, JS bundles) from git tracking - Removes
src/canvas-host/a2ui/generated files (a2ui.bundle.js and index.html) from git tracking - Updates
.gitignoreto prevent re-addition of these generated files
Reviewed changes
Copilot reviewed 1 out of 6 changed files in this pull request and generated no comments.
| File | Description |
|---|---|
.gitignore |
Adds patterns to ignore a2ui bundle files and .bundle.hash file in src/canvas-host/a2ui/ |
src/canvas-host/a2ui/index.html |
Removes 307-line generated HTML file for canvas UI |
dist/control-ui/index.html |
Removes generated HTML entry point for control UI |
dist/control-ui/assets/index-08nzABV3.css |
Removes 1-line minified CSS bundle |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
The previous commit missed the .map source map file for the control-ui bundle. This ensures all generated build artifacts are excluded from git tracking. Co-Authored-By: Claude <[email protected]>
0oAstro
force-pushed
the
fix/git-bloat-cleanup
branch
from
97bf123 to
7e83c59
Compare
Contributor
Author
|
sorry, forgot that |
Contributor
Author
|
@steipete Here are the options:
|
Member
|
@0oAstro just do a PR fixing it in the future, lets not rewrite the git history for now |
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
20 participants
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.
This PR removes generated bundle files from git tracking to reduce repo bloat.
Changes
dist/control-ui/assets (JS/CSS bundles) from gitsrc/canvas-host/a2ui/a2ui.bundle.jsandindex.htmlfrom git.serena/cache/(LSP cache from another machine)apps/**/.build/(Swift build artifacts)**/pdfjs-dist/(bundled PDF.js worker).gitignoreto prevent re-additionImpact
Size reduction (actual after gc)
Removed artifacts:
DocumentSymbols).obuild artifacts (.build/)Why this works
The files are generated at build time, not manually maintained:
pnpm ui:build(runsvite build) outputs directly todist/control-ui/(seeui/vite.config.ts)pnpm canvas:a2ui:bundlegeneratessrc/canvas-host/a2ui/a2ui.bundle.jsprepackscript ensures they're built before npm publish.o,.dia,.swiftdeps) should never be committedGenerated files that change with every build should not be committed - they bloat the git history and provide no benefit since they're regenerated anyway.