Repository navigation
docs(tunnel): LPM CLI keeps captures out of git - #391
Open
tolgaergin wants to merge 28 commits into
Open
tolgaergin wants to merge 28 commits into
tolgaergin wants to merge 28 commits into
Conversation
Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com>
Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com>
…p a committed capture Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com>
Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com>
….lpm/.gitignore Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com>
Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com>
…d git asked inside .lpm Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com>
Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com>
…tunnel refuses and clears Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com>
Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com>
…e, partial git answers and root's trust Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com>
Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com>
…answer Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com>
…ment's disk room, and shared volumes in reviews Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com>
Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com>
…elf says which repositories exist Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com>
Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com>
…s, and git stopping short reads the .git entries Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com>
Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com>
… space APFS keeps back, and how .git files are read Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com>
…s rules for .git entries, and shorter capture paragraphs Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com>
Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com>
Contributor
Author
Round 11, and the Astra and Sol reviewThe ledger is on lpm-dev/rust-client#952: lpm-dev/rust-client#952 (comment). This PR's changes (c95aecb, dates in 2eaf106):
|
…s private, and describe the path check and root's hand-over Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com>
Contributor
Author
Third review: ledgerFixed in 1b7faa1, with lpm-dev/rust-client#952 at 95c5dba7e.
|
…lish Shorter sentences, no contractions, each condition before its result, no nested list, two-item lists as prose, and the history caution's instruction first. No fact changes. Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com>
Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com>
Contributor
Author
|
Rewrote this PR's changed passages in plain English (the simple-english skill, Plain mode): sentences of at most 20 words for steps and 25 for descriptions, active voice, can/will/must instead of other modals, no contractions, semicolons or em-dashes, each condition before its result, and cautions with the instruction first. Code, identifiers, quoted messages and facts are unchanged. |
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.
Documents lpm-dev/rust-client#952. Merge with it, or once it's released.
infra/tunnel.mdx, capture history:inspector.db*to.lpm/.gitignorebefore it creates the capture database, unless the file already ignores the captures. It leaves a linked or read-only.lpm/.gitignorealone.lpm tunnelandlpm dev --tunnelmake the capture files owner-only on Unix:.lpmwritable by its owner alone, with macOS extended ACL entries removed. A database other users could read is replaced with an owner-only copy, after a damage check and a check of the disk room the copy needs, unless another process has it open. On a filesystem without Unix modes, or a macOS volume that ignores ownership, the warning says the database is open to other users.inspect,log,replayandinspect --uichange no file's access, work while a tunnel records, and warn when others could read the database, saying whether a tunnel run can change that..lpm, says when git ignores the files, names those it doesn't, and names those a repository tracks, with thegit rm --cachedcommand for each repository and the advice to rotate the credentials they held. In a repository, it names each one whose query didn't answer; outside any, it says nothing of git.log --clearoverwrites the cleared captures and empties the write-ahead log. A damaged database is moved aside..lpm/inspector.db*to.gitignore".infra/tunneling.mdx: the storage section says the same in one sentence, and thatlog --clearoverwrites the cleared captures.npm run checkpasses: content dates, SEO, lint, types, unit and component tests, and the build.🤖 Generated with Claude Code