mirror of
https://github.com/github/spec-kit.git
synced 2026-07-03 12:28:06 +08:00
* docs: add Spec Kit spec for agent-context full opt-in Use Spec Kit's own specify workflow to author the spec that makes the agent-context extension a full opt-in, removing all agent-context configuration/support from the Python codebase and removing the deprecation message. Force-added despite specs/ being gitignored; the generated artifact will be purged prior to merge. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com> * docs: add Spec Kit plan artifacts for agent-context full opt-in Phase 0/1 of the SDD plan workflow: plan.md, research.md, data-model.md, quickstart.md, and contracts/cli-behavior.md. Constitution Check is a documented no-op (repo has no ratified constitution). Force-added despite specs/ being gitignored; generated artifacts will be purged prior to merge. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com> * docs: correct Constitution Check against ratified v1.0.0 Earlier draft wrongly treated the gate as a no-op; the fork's main is 16 commits behind upstream/main, which carries .specify/memory/constitution.md. Re-evaluate the feature against Principles I-V (all PASS) and note that Principle I mandates keeping context_file as a declared class attribute, validating the R1 metadata decision. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com> * docs: refresh plan artifacts against synced upstream/main After syncing fork main to upstream and rebasing, re-scan the current agent-context surface. Upstream generalized the single context_file into a plural context_files concept with new resolver helpers (_resolve_context_files, _resolve_context_file_values, _format_context_file_values) and upsert/remove now loop over multiple files. Update research.md, data-model.md, contracts, quickstart grep guards, and the plan summary to cover the expanded removal scope. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com> * docs: add Spec Kit tasks for agent-context full opt-in Phase 2 of SDD: dependency-ordered tasks.md (30 tasks) organized by the three user stories, with mandatory test tasks (Constitution Principle II) and a foundational phase decoupling __CONTEXT_FILE__ resolution from the extension config. Includes the extension self-seeding task (T015) and a static guard test (T002) enforcing zero agent-context references in the CLI. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com> * feat!: remove agent-context lifecycle from the Specify CLI Make the agent-context extension a full opt-in. The CLI no longer installs the extension during init, writes agent-context-config.yml, or creates/updates/removes the managed Spec Kit section in agent context files. Context-section upsert/remove, marker resolution, extension-enabled gating, the config helpers, and the obsolete inline deprecation warning are all removed. Integration context_file stays as inert metadata; __CONTEXT_FILE__ now resolves from registry metadata. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com> * feat(agent-context): self-seed context file from the active integration When agent-context-config.yml has no context_file/context_files, the bundled bash and PowerShell update scripts now resolve the context file from the active integration in .specify/init-options.json via the integration registry, so the extension no longer depends on the CLI writing its config. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com> * test+docs: update suite and docs for agent-context opt-in Update integration/extension tests to expect no agent-context install, config, or context-section writes during init. Add a static guard test (test_agent_context_cli_free.py) asserting the CLI source is free of agent-context lifecycle symbols, plus backward-compatibility tests for legacy projects. Refresh AGENTS.md, the extension README, and add a CHANGELOG entry describing the opt-in behavior change. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com> * fix(agent-context): warn on self-seed failure, correct docs, speed up guard test Address PR review feedback: - Self-seed scripts (bash + PowerShell) now emit an actionable warning when an active integration is configured but specify_cli cannot be imported by the chosen Python (e.g. pipx installs), or when the integration declares no context file, instead of silently falling through to 'nothing to do'. - Correct the extension README disable note: command rendering never reads the extension config; __CONTEXT_FILE__ is always substituted from integration metadata, so a stale context_files value cannot affect rendering. - Cache CLI source reads in the static guard test via a module-scoped fixture so the directory walk happens once instead of once per forbidden symbol. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com> * feat(agent-context): ship self-owned per-agent context-file defaults The extension now bundles agent-context-defaults.json (key→context_file map) and self-seeds from it, dropping any dependency on the Specify CLI registry. Both the bash and PowerShell update scripts read the bundled JSON map keyed by the active integration from init-options.json. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com> * feat!: remove all agent-context state from the Specify CLI Strip every context_file reference from the CLI: the field on all 35 integration classes, the IntegrationBase plumbing (process_template param/step, _context_file_display, docstrings), the __CONTEXT_FILE__ resolution in agents.py, the legacy context_file/context_markers popping in _helpers.py, and the context_file template in integration_scaffold.py. Also drop the Agent context update step and __CONTEXT_FILE__ placeholder from templates/commands/plan.md. The agent-context extension now solely owns all context-file knowledge, including the per-agent default mapping. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com> * test: drop context_file coverage and guard against CLI reintroduction Remove CONTEXT_FILE attrs and context_file assertions across the base mixins, all 35 per-integration test files, shared integration tests, and conftest stubs. Rewrite the base-mixin context tests to assert no managed section is written and no __CONTEXT_FILE__ placeholder survives. Extend the CLI-free static guard to forbid context_file, __CONTEXT_FILE__, and _context_file_display in src/specify_cli, and have the extension tests copy the bundled defaults JSON so self-seed runs without the CLI. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com> * docs: reflect full removal of agent-context state from the CLI Update AGENTS.md (integration examples, required-fields table, context behavior section, pitfalls), CHANGELOG, and the SDD spec artifacts (FR-007, SC-002, data-model) to state that the CLI carries no context_file and the extension fully owns the per-agent default mapping via agent-context-defaults.json. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com> * docs: align SDD artifacts with full context_file removal Update research.md (R1, R2, R4, summary table), contracts/cli-behavior.md (C3, C5), tasks.md (Phase 2, T026, notes), plan.md (Principle I, source map), and checklists/requirements.md so the spec artifacts reflect the implemented decision: the CLI carries no context_file attribute or __CONTEXT_FILE__ resolution, and the per-agent defaults map lives in the extension. Resolves PR review #4548130110. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com> * docs: scrub stale context-file mentions from CLI docstrings Update the multi_install_safe docstring (drop the removed "context file" invariant), the RovoDev setup docstring (no longer upserts a context section), the Copilot module docstring (drop the context-file line), and tighten the _update_init_options_for_integration note. Pure docstring changes — no behavioral impact. Resolves PR review #4548237085. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com> * test+docs: harden agent-context test helper and fix stale docs - base.py: document multi_install_safe as an optional subclass attribute in the IntegrationBase docstring. - test_cli.py: clarify the init-options assertion is guarding against leftover legacy agent-context keys, not relocation. - test_extension_agent_context.py: _install_agent_context_config now asserts the bundled agent-context-defaults.json exists and always copies it, so self-seeding tests fail loudly instead of silently skipping when the map is missing. - test_integration_cursor_agent.py: drop Path/IntegrationManifest imports left unused after removing the context-section frontmatter tests. Resolves PR review #4548293116. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com> * chore: remove gitignored SDD artifacts from specs/ The specs/001-agent-context-full-optin/ artifacts were force-added for dogfooding visibility, but specs/ is gitignored and these were always intended to be purged before merge. Remove them so merging does not add an intentionally-untracked directory to repo history. Assisted-by: GitHub Copilot (model: Claude Opus 4.8, autonomous) Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com> * chore: keep CHANGELOG.md identical to upstream CHANGELOG.md is auto-generated at release time, so the branch should not carry a manual entry. Restore it to match upstream/main exactly. Assisted-by: GitHub Copilot (model: Claude Opus 4.8, autonomous) Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com> * fix: preserve Cursor .mdc frontmatter in agent-context updater scripts The bundled agent-context updater scripts wrote the managed section as plain text. For Cursor-style `.mdc` targets this dropped the required `---\nalwaysApply: true\n---` frontmatter, reintroducing the rule-loading bug originally fixed in #1699. Port the `_ensure_mdc_frontmatter` logic into both the bash and PowerShell updaters: prepend frontmatter when missing, repair `alwaysApply` when set to the wrong value, and leave non-`.mdc` targets untouched. Add regression tests covering both shells. Assisted-by: GitHub Copilot (model: Claude Opus 4.8, autonomous) Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com> * test: scope CLI-free guard to agent-context-specific symbols Drop the bare "context_file" substring from FORBIDDEN_SYMBOLS so the guard no longer fails on unrelated future CLI fields named context_file. The list still covers agent-context-specific identifiers (__CONTEXT_FILE__, _context_file_display, _resolve_context_files, _resolve_context_file_values). Assisted-by: GitHub Copilot (model: Claude Opus 4.8, autonomous) Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com> * fix: harden agent-context bash self-seed against malformed init JSON Two robustness fixes in the embedded Python self-seed logic: - Coerce the integration value from init-options.json to a string only when it is actually a string; otherwise treat it as unset so a corrupted dict/list value degrades to the existing nothing-to-do behavior instead of breaking the agents-map lookup. - Normalize agent-context-defaults.json: only use 'agents' when both the JSON root and the 'agents' value are dicts, so a wrong-shaped (but valid) JSON falls back to the warning path instead of raising on .get. Assisted-by: GitHub Copilot (model: Claude Opus 4.8, autonomous) Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com> * fix: correct PowerShell hyphenated key lookup and regex replace count - Self-seed now reads the defaults mapping via $defaults.agents.PSObject.Properties[$integrationKey].Value instead of member access ($defaults.agents.$integrationKey), which parsed hyphenated keys like 'cursor-agent'/'kiro-cli' as subtraction and failed to resolve. - Replace the static [regex]::Replace(..., 1) call, whose trailing 1 was interpreted as RegexOptions.IgnoreCase rather than a replacement count, with an instance Regex whose Replace(input, replacement, 1) limits to the first match as intended. Assisted-by: GitHub Copilot (model: Claude Opus 4.8, autonomous) Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com> * fix: make bash .mdc frontmatter guard case-insensitive The bash updater only injected Cursor .mdc frontmatter when ctx_path ended in lowercase '.mdc', so a mixed/upper-case extension (e.g. specify-rules.MDC) was skipped and Cursor would not auto-load the rule file. Compare against the casefolded path. The PowerShell variant already uses -match, which is case-insensitive by default, so no change is needed there. Assisted-by: GitHub Copilot (model: Claude Opus 4.8, autonomous) Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com> * docs: document separator-agnostic agent-context update invocation The README hard-coded the dot-notation slash command (/speckit.agent-context.update), which hyphen-separator agents like Forge and Cline do not recognize. Document the canonical command ID plus both slash invocations so users copy the form their agent accepts. Assisted-by: GitHub Copilot (model: Claude Opus 4.8, autonomous) Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com> --------- Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
508 lines
20 KiB
PowerShell
508 lines
20 KiB
PowerShell
#!/usr/bin/env pwsh
|
|
# update-agent-context.ps1
|
|
#
|
|
# Refresh the managed Spec Kit section in the coding agent's context file(s)
|
|
# (e.g. CLAUDE.md, .github/copilot-instructions.md, AGENTS.md).
|
|
#
|
|
# Reads `context_files` or `context_file`, plus `context_markers.{start,end}`, from the
|
|
# agent-context extension config:
|
|
# .specify/extensions/agent-context/agent-context-config.yml
|
|
#
|
|
# Usage: update-agent-context.ps1 [plan_path]
|
|
#
|
|
# When `plan_path` is omitted, the script derives it from `.specify/feature.json`
|
|
# (written by /speckit-specify). Falls back to the most recently modified
|
|
# `specs/*/plan.md` only when feature.json is absent or its plan does not exist yet.
|
|
|
|
[CmdletBinding()]
|
|
param(
|
|
[Parameter(Position = 0)]
|
|
[string]$PlanPath
|
|
)
|
|
|
|
function Add-MdcFrontmatter {
|
|
<#
|
|
Ensure .mdc content has YAML frontmatter with alwaysApply: true.
|
|
|
|
Cursor only auto-loads .mdc rule files that carry frontmatter with
|
|
alwaysApply: true. Prepend it when missing, or repair the value while
|
|
preserving any existing frontmatter comments/formatting.
|
|
#>
|
|
param([Parameter(Mandatory = $true)][AllowEmptyString()][string]$Content)
|
|
|
|
$leading = ''
|
|
$stripped = $Content
|
|
$m = [regex]::Match($Content, '^\s*')
|
|
if ($m.Success) {
|
|
$leading = $m.Value
|
|
$stripped = $Content.Substring($m.Length)
|
|
}
|
|
|
|
if (-not $stripped.StartsWith('---')) {
|
|
return "---`nalwaysApply: true`n---`n`n" + $Content
|
|
}
|
|
|
|
$fm = [regex]::Match($stripped, '^(---[ \t]*\r?\n)(.*?)(\r?\n---[ \t]*)(\r?\n|$)(.*)', [System.Text.RegularExpressions.RegexOptions]::Singleline)
|
|
if (-not $fm.Success) {
|
|
return "---`nalwaysApply: true`n---`n`n" + $Content
|
|
}
|
|
|
|
$opening = $fm.Groups[1].Value
|
|
$fmText = $fm.Groups[2].Value
|
|
$closing = $fm.Groups[3].Value
|
|
$sep = $fm.Groups[4].Value
|
|
$rest = $fm.Groups[5].Value
|
|
$newline = if ($opening.Contains("`r`n")) { "`r`n" } else { "`n" }
|
|
|
|
if ([regex]::IsMatch($fmText, '(?m)^[ \t]*alwaysApply[ \t]*:[ \t]*true[ \t]*(?:#.*)?$')) {
|
|
return $Content
|
|
}
|
|
|
|
if ([regex]::IsMatch($fmText, '(?m)^[ \t]*alwaysApply[ \t]*:')) {
|
|
$alwaysApplyRegex = [regex]'(?m)^([ \t]*)alwaysApply[ \t]*:.*?([ \t]*(?:#.*)?)$'
|
|
$fmText = $alwaysApplyRegex.Replace($fmText, '${1}alwaysApply: true${2}', 1)
|
|
} elseif ($fmText.Trim()) {
|
|
$fmText = $fmText + $newline + 'alwaysApply: true'
|
|
} else {
|
|
$fmText = 'alwaysApply: true'
|
|
}
|
|
|
|
return "$leading$opening$fmText$closing$sep$rest"
|
|
}
|
|
|
|
function Get-ConfigValue {
|
|
param(
|
|
[AllowNull()][object]$Object,
|
|
[Parameter(Mandatory = $true)][string]$Key
|
|
)
|
|
|
|
if ($null -eq $Object) {
|
|
return $null
|
|
}
|
|
if ($Object -is [System.Collections.IDictionary]) {
|
|
return $Object[$Key]
|
|
}
|
|
$prop = $Object.PSObject.Properties[$Key]
|
|
if ($prop) {
|
|
return $prop.Value
|
|
}
|
|
return $null
|
|
}
|
|
|
|
function Test-ConfigObject {
|
|
param(
|
|
[AllowNull()][object]$Object
|
|
)
|
|
|
|
if ($null -eq $Object) {
|
|
return $false
|
|
}
|
|
if ($Object -is [System.Collections.IDictionary]) {
|
|
return $true
|
|
}
|
|
if ($Object -is [System.Management.Automation.PSCustomObject]) {
|
|
return $true
|
|
}
|
|
return $false
|
|
}
|
|
|
|
function Resolve-ContextPath {
|
|
param(
|
|
[Parameter(Mandatory = $true)][string]$Root,
|
|
[Parameter(Mandatory = $true)][string]$RelativePath
|
|
)
|
|
|
|
$rootFull = [System.IO.Path]::GetFullPath($Root)
|
|
$segments = $RelativePath -split '/'
|
|
$resolved = $rootFull
|
|
|
|
foreach ($segment in $segments) {
|
|
if ([string]::IsNullOrWhiteSpace($segment) -or $segment -eq '.') {
|
|
continue
|
|
}
|
|
|
|
$candidate = [System.IO.Path]::GetFullPath((Join-Path $resolved $segment))
|
|
if (Test-Path -LiteralPath $candidate) {
|
|
$item = Get-Item -LiteralPath $candidate -Force
|
|
if ($item.Attributes -band [System.IO.FileAttributes]::ReparsePoint) {
|
|
$target = $item.Target
|
|
if ($target -is [System.Array]) {
|
|
$target = $target[0]
|
|
}
|
|
if ($target) {
|
|
if ([System.IO.Path]::IsPathRooted($target)) {
|
|
$candidate = [System.IO.Path]::GetFullPath($target)
|
|
} else {
|
|
$candidate = [System.IO.Path]::GetFullPath(
|
|
(Join-Path (Split-Path -Parent $candidate) $target)
|
|
)
|
|
}
|
|
}
|
|
}
|
|
}
|
|
$resolved = $candidate
|
|
}
|
|
|
|
return $resolved
|
|
}
|
|
|
|
function Test-IsSubPath {
|
|
param(
|
|
[Parameter(Mandatory = $true)][string]$Root,
|
|
[Parameter(Mandatory = $true)][string]$Path
|
|
)
|
|
|
|
$comparison = if ([System.Environment]::OSVersion.Platform -eq [System.PlatformID]::Win32NT) {
|
|
[System.StringComparison]::OrdinalIgnoreCase
|
|
} else {
|
|
[System.StringComparison]::Ordinal
|
|
}
|
|
$rootFull = [System.IO.Path]::GetFullPath($Root).TrimEnd(
|
|
[System.IO.Path]::DirectorySeparatorChar,
|
|
[System.IO.Path]::AltDirectorySeparatorChar
|
|
)
|
|
$pathFull = [System.IO.Path]::GetFullPath($Path)
|
|
return $pathFull.Equals($rootFull, $comparison) -or
|
|
$pathFull.StartsWith($rootFull + [System.IO.Path]::DirectorySeparatorChar, $comparison)
|
|
}
|
|
|
|
$ErrorActionPreference = 'Stop'
|
|
$DefaultStart = '<!-- SPECKIT START -->'
|
|
$DefaultEnd = '<!-- SPECKIT END -->'
|
|
$ProjectRoot = (Get-Location).Path
|
|
$ExtConfig = Join-Path $ProjectRoot '.specify/extensions/agent-context/agent-context-config.yml'
|
|
|
|
if (-not (Test-Path -LiteralPath $ExtConfig)) {
|
|
Write-Warning "agent-context: $ExtConfig not found; nothing to do."
|
|
exit 0
|
|
}
|
|
|
|
$Options = $null
|
|
if (Get-Command ConvertFrom-Yaml -ErrorAction SilentlyContinue) {
|
|
try {
|
|
$Options = Get-Content -LiteralPath $ExtConfig -Raw -Encoding UTF8 | ConvertFrom-Yaml -ErrorAction Stop
|
|
} catch {
|
|
# fall through to ConvertFrom-Json fallback
|
|
}
|
|
}
|
|
|
|
if ($null -eq $Options) {
|
|
# ConvertFrom-Yaml unavailable or failed; try ConvertFrom-Json (no external deps,
|
|
# works when the config file is valid JSON, which is a subset of YAML).
|
|
try {
|
|
$raw = Get-Content -LiteralPath $ExtConfig -Raw -Encoding UTF8
|
|
$Options = $raw | ConvertFrom-Json -ErrorAction Stop
|
|
if (-not (Test-ConfigObject -Object $Options)) { $Options = $null }
|
|
} catch {
|
|
$Options = $null
|
|
}
|
|
}
|
|
|
|
if ($null -eq $Options) {
|
|
# ConvertFrom-Yaml/Json unavailable or failed; fall back to Python+PyYAML.
|
|
$pythonCmd = $null
|
|
$pythonCandidates = @()
|
|
if ($env:SPECKIT_PYTHON) {
|
|
$pythonCandidates += $env:SPECKIT_PYTHON
|
|
}
|
|
$pythonCandidates += @('python3', 'python')
|
|
foreach ($candidate in $pythonCandidates) {
|
|
if (Get-Command $candidate -ErrorAction SilentlyContinue) {
|
|
# Verify it is Python 3 with PyYAML available.
|
|
$null = & $candidate -c "import sys; import yaml; sys.exit(0 if sys.version_info[0] == 3 else 1)" 2>$null
|
|
if ($LASTEXITCODE -eq 0) {
|
|
$pythonCmd = $candidate
|
|
break
|
|
}
|
|
}
|
|
}
|
|
|
|
if ($pythonCmd) {
|
|
$pyScript = $null
|
|
try {
|
|
$pyScript = [System.IO.Path]::GetTempFileName()
|
|
Set-Content -LiteralPath $pyScript -Encoding UTF8 -Value @'
|
|
import json
|
|
import sys
|
|
try:
|
|
import yaml
|
|
except ImportError:
|
|
print(
|
|
"agent-context: PyYAML is required to parse extension config; cannot update context.",
|
|
file=sys.stderr,
|
|
)
|
|
sys.exit(2)
|
|
|
|
try:
|
|
with open(sys.argv[1], "r", encoding="utf-8") as fh:
|
|
data = yaml.safe_load(fh)
|
|
except Exception as exc:
|
|
print(
|
|
f"agent-context: unable to parse {sys.argv[1]} ({exc}); cannot update context.",
|
|
file=sys.stderr,
|
|
)
|
|
sys.exit(2)
|
|
|
|
if not isinstance(data, dict):
|
|
data = {}
|
|
|
|
print(json.dumps(data))
|
|
'@
|
|
$jsonOut = & $pythonCmd $pyScript $ExtConfig
|
|
if ($LASTEXITCODE -eq 0 -and $jsonOut) {
|
|
$Options = $jsonOut | ConvertFrom-Json -ErrorAction Stop
|
|
}
|
|
} catch {
|
|
$Options = $null
|
|
} finally {
|
|
if ($pyScript -and (Test-Path -LiteralPath $pyScript)) {
|
|
Remove-Item -LiteralPath $pyScript -Force -ErrorAction SilentlyContinue
|
|
}
|
|
}
|
|
}
|
|
|
|
if (-not $Options) {
|
|
Write-Warning "agent-context: unable to parse $ExtConfig; skipping update."
|
|
exit 0
|
|
}
|
|
}
|
|
|
|
if (-not (Test-ConfigObject -Object $Options)) {
|
|
Write-Warning "agent-context: $ExtConfig must contain a YAML mapping; skipping update."
|
|
exit 0
|
|
}
|
|
|
|
$ConfiguredContextFiles = Get-ConfigValue -Object $Options -Key 'context_files'
|
|
$ContextFiles = @()
|
|
if ($null -ne $ConfiguredContextFiles) {
|
|
foreach ($item in @($ConfiguredContextFiles)) {
|
|
if ($item -is [string] -and -not [string]::IsNullOrWhiteSpace($item)) {
|
|
$ContextFiles += $item.Trim()
|
|
}
|
|
}
|
|
}
|
|
if ($ContextFiles.Count -eq 0) {
|
|
$ContextFile = Get-ConfigValue -Object $Options -Key 'context_file'
|
|
if ($ContextFile -is [string] -and -not [string]::IsNullOrWhiteSpace($ContextFile)) {
|
|
$ContextFiles += $ContextFile.Trim()
|
|
}
|
|
}
|
|
$pathComparison = if ([System.Environment]::OSVersion.Platform -eq [System.PlatformID]::Win32NT) {
|
|
[System.StringComparer]::OrdinalIgnoreCase
|
|
} else {
|
|
[System.StringComparer]::Ordinal
|
|
}
|
|
$seenContextFiles = [System.Collections.Generic.HashSet[string]]::new($pathComparison)
|
|
$dedupedContextFiles = @()
|
|
foreach ($ContextFile in $ContextFiles) {
|
|
if ($seenContextFiles.Add($ContextFile)) {
|
|
$dedupedContextFiles += $ContextFile
|
|
}
|
|
}
|
|
$ContextFiles = $dedupedContextFiles
|
|
if ($ContextFiles.Count -eq 0) {
|
|
# Self-seed: the agent-context extension owns its lifecycle, so when its
|
|
# own config declares no target it derives one from the active integration
|
|
# recorded in init-options.json, using the extension's OWN bundled mapping
|
|
# (agent-context-defaults.json). Independent of the Specify CLI by design.
|
|
$initOptionsPath = Join-Path $ProjectRoot '.specify/init-options.json'
|
|
if (Test-Path -LiteralPath $initOptionsPath) {
|
|
try {
|
|
$initOpts = Get-Content -LiteralPath $initOptionsPath -Raw | ConvertFrom-Json -ErrorAction Stop
|
|
$integrationKey = $null
|
|
if ($initOpts.PSObject.Properties['integration'] -and $initOpts.integration) {
|
|
$integrationKey = [string]$initOpts.integration
|
|
} elseif ($initOpts.PSObject.Properties['ai'] -and $initOpts.ai) {
|
|
$integrationKey = [string]$initOpts.ai
|
|
}
|
|
if ($integrationKey) {
|
|
$defaultsPath = Join-Path $ProjectRoot '.specify/extensions/agent-context/agent-context-defaults.json'
|
|
if (Test-Path -LiteralPath $defaultsPath) {
|
|
$defaults = Get-Content -LiteralPath $defaultsPath -Raw | ConvertFrom-Json -ErrorAction Stop
|
|
$derived = $null
|
|
if ($defaults.PSObject.Properties['agents'] -and $defaults.agents.PSObject.Properties[$integrationKey]) {
|
|
$derived = [string]$defaults.agents.PSObject.Properties[$integrationKey].Value
|
|
}
|
|
if ($derived -and -not [string]::IsNullOrWhiteSpace($derived)) {
|
|
$ContextFiles += $derived.Trim()
|
|
} else {
|
|
Write-Warning ("agent-context: no default context file is known for integration '{0}'; set 'context_file' in the extension config to choose one." -f $integrationKey)
|
|
}
|
|
} else {
|
|
Write-Warning ("agent-context: unable to read {0}; cannot self-seed the context file. Set 'context_file' in the extension config." -f $defaultsPath)
|
|
}
|
|
}
|
|
} catch {
|
|
# Non-fatal: fall through to the nothing-to-do guard below.
|
|
}
|
|
}
|
|
}
|
|
if ($ContextFiles.Count -eq 0) {
|
|
Write-Warning 'agent-context: context_files/context_file not set in extension config; nothing to do.'
|
|
exit 0
|
|
}
|
|
|
|
foreach ($ContextFile in $ContextFiles) {
|
|
# Reject absolute paths, drive-qualified paths, backslash separators, and '..' path segments in context files
|
|
if ($ContextFile -match '^[A-Za-z]:') {
|
|
Write-Warning "agent-context: context files must be project-relative paths; got '$ContextFile'."
|
|
exit 1
|
|
}
|
|
if ([System.IO.Path]::IsPathRooted($ContextFile)) {
|
|
Write-Warning "agent-context: context files must be project-relative paths; got '$ContextFile'."
|
|
exit 1
|
|
}
|
|
if ($ContextFile.Contains('\')) {
|
|
Write-Warning "agent-context: context files must not contain backslash separators; got '$ContextFile'."
|
|
exit 1
|
|
}
|
|
$cfSegments = $ContextFile -split '[/\\]'
|
|
if ($cfSegments -contains '..') {
|
|
Write-Warning "agent-context: context files must not contain '..' path segments; got '$ContextFile'."
|
|
exit 1
|
|
}
|
|
$resolvedTarget = Resolve-ContextPath -Root $ProjectRoot -RelativePath $ContextFile
|
|
if (-not (Test-IsSubPath -Root $ProjectRoot -Path $resolvedTarget)) {
|
|
Write-Warning "agent-context: context file path resolves outside the project root; got '$ContextFile'."
|
|
exit 1
|
|
}
|
|
}
|
|
|
|
$MarkerStart = $DefaultStart
|
|
$MarkerEnd = $DefaultEnd
|
|
$cm = Get-ConfigValue -Object $Options -Key 'context_markers'
|
|
if ($cm) {
|
|
$cmStart = Get-ConfigValue -Object $cm -Key 'start'
|
|
if ($cmStart -is [string] -and $cmStart) {
|
|
$MarkerStart = $cmStart
|
|
}
|
|
$cmEnd = Get-ConfigValue -Object $cm -Key 'end'
|
|
if ($cmEnd -is [string] -and $cmEnd) {
|
|
$MarkerEnd = $cmEnd
|
|
}
|
|
}
|
|
|
|
if (-not $PlanPath) {
|
|
# Prefer .specify/feature.json (written by /speckit-specify) over mtime heuristic.
|
|
$FeatureJson = Join-Path $ProjectRoot '.specify/feature.json'
|
|
if (Test-Path -LiteralPath $FeatureJson) {
|
|
try {
|
|
$fj = Get-Content -LiteralPath $FeatureJson -Raw -Encoding UTF8 | ConvertFrom-Json
|
|
$featureDir = $fj.feature_directory
|
|
if ($featureDir -isnot [string] -or -not $featureDir) {
|
|
$featureDir = $null
|
|
} else {
|
|
$featureDir = $featureDir.TrimEnd('\', '/')
|
|
}
|
|
if ($featureDir) {
|
|
# Join-Path on Unix does not treat absolute ChildPath as "wins"; check explicitly.
|
|
if ([System.IO.Path]::IsPathRooted($featureDir)) {
|
|
$candidatePlan = Join-Path $featureDir 'plan.md'
|
|
} else {
|
|
$candidatePlan = Join-Path (Join-Path $ProjectRoot $featureDir) 'plan.md'
|
|
}
|
|
if (Test-Path -LiteralPath $candidatePlan) {
|
|
# Resolve ./ .. segments before relativizing (mirrors bash Path.resolve()).
|
|
# GetFullPath is available in .NET Framework 4.x (PS 5.1 compatible).
|
|
$resolvedPlan = [System.IO.Path]::GetFullPath($candidatePlan)
|
|
$resolvedDir = [System.IO.Path]::GetDirectoryName($resolvedPlan)
|
|
$normRoot = $ProjectRoot.TrimEnd('\', '/') + [System.IO.Path]::DirectorySeparatorChar
|
|
$normDir = $resolvedDir.TrimEnd('\', '/') + [System.IO.Path]::DirectorySeparatorChar
|
|
$cmp = if ([System.Environment]::OSVersion.Platform -eq [System.PlatformID]::Win32NT) { [System.StringComparison]::OrdinalIgnoreCase } else { [System.StringComparison]::Ordinal }
|
|
if ($normDir.StartsWith($normRoot, $cmp)) {
|
|
$relDir = $normDir.Substring($normRoot.Length).TrimEnd('\', '/')
|
|
$PlanPath = if ($relDir) { $relDir.Replace('\', '/') + '/plan.md' } else { 'plan.md' }
|
|
} else {
|
|
$PlanPath = $resolvedPlan.Replace('\', '/')
|
|
}
|
|
}
|
|
}
|
|
} catch {
|
|
# Non-fatal: fall through to mtime heuristic.
|
|
}
|
|
}
|
|
|
|
# Fall back to mtime only when feature.json is absent or its plan does not exist yet.
|
|
if (-not $PlanPath) {
|
|
try {
|
|
$specsDir = Join-Path $ProjectRoot 'specs'
|
|
$candidate = Get-ChildItem -Path $specsDir -Directory -ErrorAction SilentlyContinue |
|
|
ForEach-Object { Get-Item -LiteralPath (Join-Path $_.FullName 'plan.md') -ErrorAction SilentlyContinue } |
|
|
Where-Object { $_ } |
|
|
Sort-Object LastWriteTime -Descending |
|
|
Select-Object -First 1
|
|
if ($candidate) {
|
|
# GetRelativePath is .NET 5+ only; strip prefix manually for PS 5.1 compat.
|
|
# Use case-insensitive comparison on Windows only (matches common.ps1 pattern).
|
|
$fullPath = $candidate.FullName.Replace('\', '/')
|
|
$normRoot = $ProjectRoot.Replace('\', '/').TrimEnd('/') + '/'
|
|
$cmp = if ([System.Environment]::OSVersion.Platform -eq [System.PlatformID]::Win32NT) { [System.StringComparison]::OrdinalIgnoreCase } else { [System.StringComparison]::Ordinal }
|
|
if ($fullPath.StartsWith($normRoot, $cmp)) {
|
|
$PlanPath = $fullPath.Substring($normRoot.Length)
|
|
} else {
|
|
$PlanPath = $fullPath
|
|
}
|
|
}
|
|
} catch {
|
|
# Non-fatal: continue without a plan path.
|
|
}
|
|
}
|
|
}
|
|
|
|
$lines = @($MarkerStart,
|
|
'For additional context about technologies to be used, project structure,',
|
|
'shell commands, and other important information, read the current plan')
|
|
if ($PlanPath) {
|
|
$lines += "at $PlanPath"
|
|
}
|
|
$lines += $MarkerEnd
|
|
$Section = ($lines -join "`n") + "`n"
|
|
|
|
foreach ($ContextFile in $ContextFiles) {
|
|
$CtxPath = Join-Path $ProjectRoot $ContextFile
|
|
$CtxDir = Split-Path -Parent $CtxPath
|
|
if ($CtxDir -and -not (Test-Path -LiteralPath $CtxDir)) {
|
|
New-Item -ItemType Directory -Path $CtxDir -Force | Out-Null
|
|
}
|
|
|
|
if (Test-Path -LiteralPath $CtxPath) {
|
|
$rawBytes = [System.IO.File]::ReadAllBytes($CtxPath)
|
|
# Strip UTF-8 BOM if present
|
|
if ($rawBytes.Length -ge 3 -and $rawBytes[0] -eq 0xEF -and $rawBytes[1] -eq 0xBB -and $rawBytes[2] -eq 0xBF) {
|
|
$content = [System.Text.Encoding]::UTF8.GetString($rawBytes, 3, $rawBytes.Length - 3)
|
|
} else {
|
|
$content = [System.Text.Encoding]::UTF8.GetString($rawBytes)
|
|
}
|
|
|
|
$s = $content.IndexOf($MarkerStart)
|
|
$e = if ($s -ge 0) { $content.IndexOf($MarkerEnd, $s) } else { $content.IndexOf($MarkerEnd) }
|
|
|
|
if ($s -ge 0 -and $e -ge 0 -and $e -gt $s) {
|
|
$endOfMarker = $e + $MarkerEnd.Length
|
|
if ($endOfMarker -lt $content.Length -and $content[$endOfMarker] -eq "`r") { $endOfMarker++ }
|
|
if ($endOfMarker -lt $content.Length -and $content[$endOfMarker] -eq "`n") { $endOfMarker++ }
|
|
$newContent = $content.Substring(0, $s) + $Section + $content.Substring($endOfMarker)
|
|
} elseif ($s -ge 0) {
|
|
$newContent = $content.Substring(0, $s) + $Section
|
|
} elseif ($e -ge 0) {
|
|
$endOfMarker = $e + $MarkerEnd.Length
|
|
if ($endOfMarker -lt $content.Length -and $content[$endOfMarker] -eq "`r") { $endOfMarker++ }
|
|
if ($endOfMarker -lt $content.Length -and $content[$endOfMarker] -eq "`n") { $endOfMarker++ }
|
|
$newContent = $Section + $content.Substring($endOfMarker)
|
|
} else {
|
|
if ($content -and -not $content.EndsWith("`n")) { $content += "`n" }
|
|
if ($content) { $newContent = $content + "`n" + $Section } else { $newContent = $Section }
|
|
}
|
|
} else {
|
|
$newContent = $Section
|
|
}
|
|
|
|
$newContent = $newContent.Replace("`r`n", "`n").Replace("`r", "`n")
|
|
if ($ContextFile -match '\.mdc$') {
|
|
$newContent = Add-MdcFrontmatter -Content $newContent
|
|
}
|
|
[System.IO.File]::WriteAllText($CtxPath, $newContent, (New-Object System.Text.UTF8Encoding($false)))
|
|
|
|
Write-Host "agent-context: updated $ContextFile"
|
|
}
|