Skip to content

Test: Verify CLA Assistant Bot#346

Closed
Tarquinen wants to merge 74 commits intomasterfrom
test-cla
Closed

Test: Verify CLA Assistant Bot#346
Tarquinen wants to merge 74 commits intomasterfrom
test-cla

Conversation

@Tarquinen
Copy link
Collaborator

This PR is created to verify that the CLA Assistant bot is correctly configured and responding to new Pull Requests.

Tarquinen and others added 30 commits January 29, 2026 20:16
Recover squash-related files from dangling stash commit:
- squash tool implementation and utils
- squash, discard-extract, discard-squash, extract-squash, all prompt variants
- system and nudge prompts for all tool combinations
- Add ulid dependency for generating unique IDs
- Replace timestamp-based ID counter with ulid
- Ensures unique IDs across parallel operations
- Move squashSummaries out of prune object to top-level state
- Add resetOnCompaction utility to clear stale state after compaction
- Move findLastCompactionTimestamp and countTurns to state/utils.ts
- Update isMessageCompacted to also check prune.messageIds
- Update persistence to save/load squashSummaries at top level
- Move findStringInMessages, collectToolIdsInRange, collectMessageIdsInRange
  from squash.ts to tools/utils.ts for reusability
- Keep squash.ts focused on tool definition and execution logic
- Follows existing pattern where collectContentInRange lives in utils.ts
When opencode runs with OPENCODE_SERVER_PASSWORD set, the server
requires HTTP Basic Auth. This adds auth utilities to detect secure
mode and configure the SDK client with an interceptor that injects
the Authorization header on all requests.

Fixes #304
- revert terse PR 313 style back to verbose prose format from dev branch
- clarify tool triggers: discard/extract for individual outputs, squash for phases
- remove 'task complete' trigger from discard/extract (exclusive to squash)
- add instruction to not use discard/extract without prunable-tools list
- replace 'task/sub-task' terminology with 'phase' to avoid conflict with Task tool
- Use tool part injection for DeepSeek/Kimi (requires reasoning_content in assistant messages)
- Use text part injection for other models (cleaner approach)
- Add prunedMessageCount to context breakdown display
refactor(prompts): dcp prompt system + prompt dx cli + new tool names
Fixes issue where readFileSync with __dirname fails when bundled by Bun
(same issue as #222, reintroduced by #327).

- Add scripts/generate-prompts.ts prebuild script
- Import generated .ts files instead of runtime readFileSync
- Remove postbuild .md copy (no longer needed)
Tarquinen and others added 27 commits January 31, 2026 20:37
- Change toolIds and messageIds from string[] to Set<string>
- Update all .includes() to .has(), .push() to .add(), .length to .size
- Add serialization layer in persistence.ts for JSON compatibility
- Remove redundant Set wrappers now that state is already a Set
- Fix session initialization order in dcp commands
- Fix token calculation for pruned tools
feat: support toast notifications via pruneNotificationType
Remove hybrid injection strategy that used synthetic assistant messages
for most models while special-casing DeepSeek/Kimi with tool parts.
Now uses tool part injection universally when last message is assistant.
Python CLI tools for analyzing OpenCode session data:
- opencode-dcp-stats: DCP cache impact analysis
- opencode-find-session: find sessions by title
- opencode-session-timeline: per-step token timeline
- opencode-token-stats: aggregate token usage stats
Remove model-specific branching (DeepSeek/Kimi check) and unused helper
functions. Now all non-user-message injections use synthetic tool parts
appended to the last message, eliminating ~100 lines of code.
@CLAassistant
Copy link

CLAassistant commented Feb 4, 2026

CLA assistant check
Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you all sign our Contributor License Agreement before we can accept your contribution.
1 out of 3 committers have signed the CLA.

✅ Tarquinen
❌ spoons-and-mirrors
❌ essinghigh
You have signed the CLA already but the status is still pending? Let us recheck it.

@Tarquinen Tarquinen closed this Feb 4, 2026
@Tarquinen Tarquinen deleted the test-cla branch February 10, 2026 16:37
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants

Comments