chore(ci): bump pnpm/action-setup from 4 to 6#3
Merged
Conversation
Author
LabelsThe following labels could not be found: Please fix the above issues or remove invalid values from |
|
Important Review skippedBot user detected. To trigger a single review, invoke the ⚙️ Run configurationConfiguration used: defaults Review profile: CHILL Plan: Free Run ID: You can disable this status message by setting the Use the checkbox below for a quick retry:
Comment |
Contributor
|
@dependabot rebase |
Bumps [pnpm/action-setup](https://github.com/pnpm/action-setup) from 4 to 6. - [Release notes](https://github.com/pnpm/action-setup/releases) - [Commits](pnpm/action-setup@v4...v6) --- updated-dependencies: - dependency-name: pnpm/action-setup dependency-version: '6' dependency-type: direct:production update-type: version-update:semver-major ... Signed-off-by: dependabot[bot] <support@github.com>
dependabot
Bot
force-pushed
the
dependabot/github_actions/pnpm/action-setup-6
branch
from
May 8, 2026 04:28
ea6d15a to
cbd93b5
Compare
This was referenced Jun 17, 2026
6 tasks
thewrz
added a commit
that referenced
this pull request
Jul 1, 2026
Addresses live-review feedback on the demo's coordination audit view and chat sidebar. - Full collapse (#2): a collapsed Report category/section now fully hides — the padded leaf list zeroes its vertical padding when closed and the two new intermediate tree wrappers are padding-free, so no ~1rem sliver remains. - Tree hierarchy (A): the Report is a 3-level collapsible tree (category -> CSI division -> section), independently foldable at each level, so very large projects stay navigable. Reuses the existing chevron / aria-expanded / grid-rows collapse primitive via a shared collapsibleShell. - Locate + highlight (#1): section-level findings (no paragraph anchor) now highlight the section head instead of silently scrolling, so every finding category gives visible feedback; paragraph-level findings are unchanged. (Paragraph-precise anchors for the 3 reference finding types need a backend change and are tracked separately.) - Implied-related (#3): hidden behind its feature flag pending a fix for a false-positive inference (single "control" token). See #327. - Chat history (I): the sidebar conversation persists in localStorage (survives reloads), sends only a bounded recent window to the model (never trips the server's 40-message cap), and gains a Clear control. Mirrors WrzDJ's full-transcript-vs-bounded-context split. Verified in-browser (Playwright): 3-level tree renders (10 categories -> 13 divisions -> 13 sections), collapse 775->0 / 242->0 with no sliver, section- head and paragraph highlights fire, implied category absent, chat replays on reload and Clear wipes it. 0 console errors. Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
thewrz
added a commit
that referenced
this pull request
Jul 2, 2026
Addresses live-review feedback on the demo's coordination audit view and chat sidebar. - Full collapse (#2): a collapsed Report category/section now fully hides — the padded leaf list zeroes its vertical padding when closed and the two new intermediate tree wrappers are padding-free, so no ~1rem sliver remains. - Tree hierarchy (A): the Report is a 3-level collapsible tree (category -> CSI division -> section), independently foldable at each level, so very large projects stay navigable. Reuses the existing chevron / aria-expanded / grid-rows collapse primitive via a shared collapsibleShell. - Locate + highlight (#1): section-level findings (no paragraph anchor) now highlight the section head instead of silently scrolling, so every finding category gives visible feedback; paragraph-level findings are unchanged. (Paragraph-precise anchors for the 3 reference finding types need a backend change and are tracked separately.) - Implied-related (#3): hidden behind its feature flag pending a fix for a false-positive inference (single "control" token). See #327. - Chat history (I): the sidebar conversation persists in localStorage (survives reloads), sends only a bounded recent window to the model (never trips the server's 40-message cap), and gains a Clear control. Mirrors WrzDJ's full-transcript-vs-bounded-context split. Verified in-browser (Playwright): 3-level tree renders (10 categories -> 13 divisions -> 13 sections), collapse 775->0 / 242->0 with no sliver, section- head and paragraph highlights fire, implied category absent, chat replays on reload and Clear wipes it. 0 console errors. Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
thewrz
added a commit
that referenced
this pull request
Jul 8, 2026
… env isolation, docs - file-loader: collect parseWarnings before the dry-run early return so a preview surfaces warnings in LoadResult.parseWarnings, not just the log (Major #3) + regression test pinning the dry-run path. - log-context.test: snapshot/restore NODE_ENV/DATABASE_URL in one file-level beforeAll/afterAll instead of two unrestored beforeAll hooks — no env leak into later Vitest files in the same worker (#4). - ADR-056: LOG_FILE -> LOG_TO_FILE (the actual env gate) in Decision 1 + Inv 5 (#1). - plan: logger-options test path -> logger.test.ts (matches shipped code) and reorder the processFile sample to match the corrected dry-run behavior (#2/#3). Refs #422 Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
thewrz
added a commit
that referenced
this pull request
Jul 8, 2026
Address two CodeRabbit nitpicks on PR #423 and record a third as deferred: - #8 (logger.ts): the pino-roll file transport runs on a worker thread that buffers writes, so the bare process.exit(0) in shutdown() could kill it before the buffer flushed — dropping the final JSONL lines the corpus fix-loop reads back. Build the transport explicitly so we hold its handle, and drain it via closeLogger() (awaited in index.ts shutdown, guarded to always terminate). No-op when logging to stdout. Pinned by a unit test. - #7 (ADR-056): document the retention/redaction posture now that the user-supplied filename lands in durable JSONL — bounded by pino-roll rotation, namespaced under an app-controlled `doc` key, not redacted by default (spec filenames are not secrets), with a redact path noted for deployments that disagree. - #3 (ADR-056 P2): recorded as the first deferred item — carry the caught error's cause/detail on ParseWarning so core-metadata-unreadable surfaces *why* it failed, not just *that* it did. Deferred as an AST/contract change beyond this PR's boundary-only scope. The other 6 nitpicks are correctly skippable (one, the error.ts `declare override`, does not compile under useDefineForClassFields — the load-bearing `= undefined` initializer is documented in the Task 4 gotcha). Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
This was referenced Jul 11, 2026
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
1 participant
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.
Bumps pnpm/action-setup from 4 to 6.
Release notes
Sourced from pnpm/action-setup's releases.
Commits
8912a91fix: append (not prepend) action node dir to PATH for npm bootstrap (#241)26f6d4ffix: use npm co-located with the action node binary (#239)903f9c1fix: update pnpm to 11.0.0-rc.5bdf0af2test: add strict version-match jobs to reproduce #225 / #22771c9247fix: pnpm self-update binary shadowed by bootstrap on PATH (#230)078e9d4fix: update pnpm to 11.0.0-rc.208c4be7docs(README): update action-setup version5798914chore: update .gitignoreddffd66fix: remove accidentally committed fileb43f991fix: update pnpm to 11.0.0-rc.0