Fix auto-briefing path normalization, fingerprint caching, and doc annotation artifacts

[Copilot]

Three bugs from PR review: relative editedFilePath values from file.edited events silently returned empty seed IDs; computeBriefIntent() re-read symbols.yaml from disk on every system.transform invocation; and accidental #YT|> prefixes were left in three v1 documentation files.

Changes

• brief-intent.ts — path normalization: deriveSeedIds() was passing editedFilePath directly to getSourceLinkedRequirementIds(), which calls path.relative(worktree, editedAbsolutePath) and expects an absolute path. Relative paths (typical from file.edited) never matched symbol entries, silently returning []. Now joins with workspaceRoot when not absolute; sourceFiles still preserves the original value.

  const absoluteEditedPath = path.isAbsolute(params.editedFilePath)
    ? params.editedFilePath
    : path.join(params.workspaceRoot, params.editedFilePath);

• index.ts — fingerprint caching: system.transform was reconstructing a computeBriefIntent() result solely to get a fingerprint for autoBriefResults lookup. That call synchronously parses symbols.yaml on every transform. Now stores lastBriefFingerprint when the intent is computed in the file.edited handler and uses it directly in the transform hook.

• Documentation cleanup: Removed #YT|> annotation prefixes from deprecated-note lines in REQ-opencode-kibi-briefing-v1.md, SCEN-opencode-kibi-briefing-v1.md, and TEST-opencode-kibi-briefing-v1.md.

[Copilot]

Pull request overview

This PR refines OpenCode’s auto-briefing v2 flow by simplifying how the prompt hook retrieves the most recent auto-brief result, normalizing edited-file path handling for source-linked seed IDs, and cleaning up deprecated v1 documentation note formatting.

Changes:

• Track the last computed brief “fingerprint” from file edits and use it directly in the system prompt transform hook.
• Normalize editedFilePath to an absolute path before resolving source-linked requirement seed IDs.
• Reformat deprecated v1 briefing docs “Note” blockquotes.

Reviewed changes

Copilot reviewed 5 out of 5 changed files in this pull request and generated 5 comments.

Show a summary per file

File	Description
packages/opencode/src/index.ts	Stores lastBriefFingerprint from edit events and uses it to retrieve cached auto-brief results during prompt injection.
packages/opencode/src/brief-intent.ts	Ensures getSourceLinkedRequirementIds receives an absolute edited file path for consistent lookups.
documentation/tests/TEST-opencode-kibi-briefing-v1.md	Reformats deprecated note blockquote.
documentation/scenarios/SCEN-opencode-kibi-briefing-v1.md	Reformats deprecated note blockquote.
documentation/requirements/REQ-opencode-kibi-briefing-v1.md	Reformats deprecated note blockquote.

[Copilot]

lastBriefFingerprint is only updated inside the isAutoBriefRisk branch, but the system.transform hook always uses it to fetch autoBriefResult. After a subsequent non-auto-brief edit (e.g. safe_docs_only / safe_test_only), this can keep showing a stale auto-brief result from an earlier risky edit. Consider clearing lastBriefFingerprint whenever an edit is processed that won’t compute/fetch a new briefing (or updating it for every edit), so the guidance reflects the current context.

Suggested change

	const autoBriefResult = lastBriefFingerprint != null
	? autoBriefResults.get(lastBriefFingerprint)
	: undefined;
	const shouldUseAutoBriefResult =
	lastRiskClass !== "safe_docs_only" &&
	lastRiskClass !== "safe_test_only";
	if (!shouldUseAutoBriefResult) {
	lastBriefFingerprint = undefined;
	}
	const autoBriefResult =
	shouldUseAutoBriefResult && lastBriefFingerprint != null
	? autoBriefResults.get(lastBriefFingerprint)
	: undefined;

[Copilot]

lastEditedFilePath is now assigned but never read (it was previously only used to recompute the auto-brief fingerprint). This looks like leftover state and will likely trigger an unused-variable lint/TS error; remove it or reintroduce a use that matches the intended behavior.

Suggested change

let lastEditedFilePath: string | null = null;

[Copilot]

This line ends with trailing whitespace. The repo’s .editorconfig sets trim_trailing_whitespace = true for *.md, so please remove the extra space at the end of the line.

Suggested change

	> **Note**: This test doc is DEPRECATED and superseded by TEST-opencode-kibi-briefing-v2.
	> **Note**: This test doc is DEPRECATED and superseded by TEST-opencode-kibi-briefing-v2.

[Copilot]

This line ends with trailing whitespace. The repo’s .editorconfig sets trim_trailing_whitespace = true for *.md, so please remove the extra space at the end of the line.

Suggested change

	> **Note**: This scenario is DEPRECATED and superseded by SCEN-opencode-kibi-briefing-v2.
	> **Note**: This scenario is DEPRECATED and superseded by SCEN-opencode-kibi-briefing-v2.

[Copilot]

This line ends with trailing whitespace. The repo’s .editorconfig sets trim_trailing_whitespace = true for *.md, so please remove the extra space at the end of the line.

Suggested change

	> **Note**: This requirement is DEPRECATED and superseded by REQ-opencode-kibi-briefing-v2.
	> **Note**: This requirement is DEPRECATED and superseded by REQ-opencode-kibi-briefing-v2.