Skip to content

skill(doublecheck): run in forked subagent context#1578

Open
wisenbergg wants to merge 2 commits intogithub:stagedfrom
wisenbergg:doublecheck-context-fork
Open

skill(doublecheck): run in forked subagent context#1578
wisenbergg wants to merge 2 commits intogithub:stagedfrom
wisenbergg:doublecheck-context-fork

Conversation

@wisenbergg
Copy link
Copy Markdown

The doublecheck skill runs a 3-layer verification pipeline (claim extraction, web source search, adversarial review) that pulls in substantial reference material. Running it inline pollutes the main chat context with intermediate fetches and lowers the quality of follow-up responses.

Setting context: fork isolates execution in a dedicated subagent. The structured verification report still returns to the main chat, but the intermediate noise stays out of it. Same pattern already used by microsoft-skill-creator.

Changes: 1-line frontmatter addition in:

  • skills/doublecheck/SKILL.md
  • plugins/doublecheck/skills/doublecheck/SKILL.md

Copilot AI review requested due to automatic review settings May 1, 2026 05:07
@wisenbergg wisenbergg requested a review from dvelton as a code owner May 1, 2026 05:07
Copy link
Copy Markdown
Contributor

@github-actions github-actions Bot left a comment

Choose a reason for hiding this comment

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

⚠️ This PR targets main, but PRs should target staged.

The main branch is auto-published from staged and should not receive direct PRs.
Please close this PR and re-open it against the staged branch.

You can change the base branch using the Edit button at the top of this PR,
or run: gh pr edit 1578 --base staged

@wisenbergg wisenbergg changed the base branch from main to staged May 1, 2026 05:08
@wisenbergg wisenbergg requested a review from aaronpowell as a code owner May 1, 2026 05:08
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Copilot wasn't able to review this pull request because it exceeds the maximum number of files (300). Try reducing the number of changed files and requesting a review from Copilot again.

@aaronpowell
Copy link
Copy Markdown
Contributor

It looks like you've incorrectly branched from the main branch not staged, and as a result all the materialised plugins are included in this PR.

You can attempt to fix this with a rebase:

git fetch origin staged
git rebase --onto origin/staged origin/main <branch name>
git push --force-with-lease

If that does not resolve it, you can run npm run plugin:clean which will delete the materialised plugins and you can commit that change.

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.

3 participants