Skip to content

Optional learning pass prior to compression - #594

Open
Malkovsky wants to merge 1 commit into
Opencode-DCP:masterfrom
Malkovsky:feature/compression-learning
Open

Optional learning pass prior to compression#594
Malkovsky wants to merge 1 commit into
Opencode-DCP:masterfrom
Malkovsky:feature/compression-learning

Conversation

@Malkovsky

@Malkovsky Malkovsky commented Jul 31, 2026

Copy link
Copy Markdown

Introduces optional learning pass right before the compression.

Problem

Many projects dedicate a memory mechanisms for the agents in a different forms: AGENTS.md, project scoped skills, examples etc. Compression and learning are usually separated mechanisms, the problem is that compression might prune some useful insights about the project.

Solution

Add an optional parameter "learning" in the DCP config, disabled by default. When enabled injects the prompt to perform learning based on the project rules (prompt is similar to opencode "learn" command).

AI disclosure

Configs and tests are implemented by GPT-5.6 Sol

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: d72fb98f53

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

Comment thread dcp.schema.json
"type": "object",
"description": "Optional learning pass before compression that extracts durable codebase knowledge",
"additionalProperties": false,
"required": ["enabled", "notifications"],

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

P2 Badge Allow partial learning overrides

When a user sets only compress.learning.enabled, or a project layer overrides only notifications, this requirement makes an otherwise usable layered configuration schema-invalid; validateConfigTypes likewise reports the omitted sibling as having type undefined, even though mergeCompress explicitly supplies both missing values from the previous/default layer. Remove the requirement and validate each property only when present so users can override these settings independently like the other nested configuration fields.

Useful? React with 👍 / 👎.

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.

1 participant