Skip to content

Add regression coverage for agent-context symlink updates#3017

Open
Jiandong7 wants to merge 1 commit into
github:mainfrom
Jiandong7:test/1464-agent-context-symlink
Open

Add regression coverage for agent-context symlink updates#3017
Jiandong7 wants to merge 1 commit into
github:mainfrom
Jiandong7:test/1464-agent-context-symlink

Conversation

@Jiandong7

Copy link
Copy Markdown
Contributor

Summary

  • add regression coverage for update-agent-context.sh when the configured context file is a symlink
  • assert the symlink is preserved and the managed section is written through to the target file

Validation

  • python -m pytest tests\extensions\test_extension_agent_context.py -q (34 passed, 1 skipped locally because working bash is unavailable)
  • git diff --check

Addresses #1464

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Copilot encountered an error and was unable to review this pull request. You can try again by re-requesting a review.

@Jiandong7 Jiandong7 force-pushed the test/1464-agent-context-symlink branch from 41a35f4 to f100c9d Compare June 24, 2026 01:27
@mnriem mnriem requested a review from Copilot June 24, 2026 16:05

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Copilot's findings

  • Files reviewed: 1/1 changed files
  • Comments generated: 1

Comment on lines +126 to +135
result = subprocess.run(
[
BASH,
str(EXT_DIR / "scripts" / "bash" / "update-agent-context.sh"),
"specs/001-demo/plan.md",
],
cwd=tmp_path,
text=True,
capture_output=True,
)
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