Skip to content

fix: update test snapshots after AGENTS.md refactoring#2

Merged
Eric-Guo merged 2 commits intoagends_md_as_first_messagefrom
copilot/sub-pr-1
Nov 15, 2025
Merged

fix: update test snapshots after AGENTS.md refactoring#2
Eric-Guo merged 2 commits intoagends_md_as_first_messagefrom
copilot/sub-pr-1

Conversation

Copy link

Copilot AI commented Nov 15, 2025

Related Issue

Resolve #(issue_number)

Description

The AGENTS.md refactoring that moved project context from system prompt to first assistant message was already implemented, but test snapshots were not updated to reflect the change.

Changes:

  • Fixed code formatting in kimisoul.py (f-string line continuation)
  • Updated test snapshots in test_default_agent.py and test_task_subagents.py to remove expected "# Project Information" section from system prompt
  • Tests now correctly verify that AGENTS.md content is injected via _ensure_initial_system_messages() rather than baked into the system prompt template

The refactoring moves AGENTS.md from BuiltinSystemPromptArgs (removed KIMI_AGENTS_MD) to Runtime.agents_md, which is loaded once at startup and injected as an assistant message with <system> tags on first run with empty context.


💬 We'd love your input! Share your thoughts on Copilot coding agent in our 2 minute survey.

…ting

Co-authored-by: Eric-Guo <1131536+Eric-Guo@users.noreply.github.com>
Copy link
Author

Copilot AI commented Nov 15, 2025

Just as a heads up, I was blocked by some firewall rules while working on your feedback. Expand below for details.

Warning

Firewall rules blocked me from connecting to one or more addresses (expand for details)

I tried to connect to the following addresses, but was blocked by firewall rules:

  • astral.sh
    • Triggering command: curl -LsSf REDACTED (dns block)
  • this-domain-definitely-does-not-exist-12345.com
    • Triggering command: /home/REDACTED/work/kimi-cli/kimi-cli/.venv/bin/python /home/REDACTED/work/kimi-cli/kimi-cli/.venv/bin/pytest tests -vv (dns block)
  • www.moonshot.ai
    • Triggering command: /home/REDACTED/work/kimi-cli/kimi-cli/.venv/bin/python /home/REDACTED/work/kimi-cli/kimi-cli/.venv/bin/pytest tests -vv (dns block)

If you need me to access, download, or install something from one of these locations, you can either:

Copilot AI changed the title [WIP] Refactor to move agents.md to first message from system prompt fix: update test snapshots after AGENTS.md refactoring Nov 15, 2025
Copilot AI requested a review from Eric-Guo November 15, 2025 03:46
@Eric-Guo Eric-Guo marked this pull request as ready for review November 15, 2025 03:47
@Eric-Guo Eric-Guo merged commit 9e274d0 into agends_md_as_first_message Nov 15, 2025
@Eric-Guo Eric-Guo deleted the copilot/sub-pr-1 branch November 15, 2025 03:51
Eric-Guo added a commit that referenced this pull request Nov 17, 2025
* Initial plan

* fix: update test snapshots after AGENTS.md refactoring and fix formatting

Co-authored-by: Eric-Guo <1131536+Eric-Guo@users.noreply.github.com>

---------

Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com>
Co-authored-by: Eric-Guo <1131536+Eric-Guo@users.noreply.github.com>
Eric-Guo added a commit that referenced this pull request Nov 18, 2025
* Initial plan

* fix: update test snapshots after AGENTS.md refactoring and fix formatting

Co-authored-by: Eric-Guo <1131536+Eric-Guo@users.noreply.github.com>

---------

Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com>
Co-authored-by: Eric-Guo <1131536+Eric-Guo@users.noreply.github.com>
Eric-Guo added a commit that referenced this pull request Nov 20, 2025
* Initial plan

* fix: update test snapshots after AGENTS.md refactoring and fix formatting

Co-authored-by: Eric-Guo <1131536+Eric-Guo@users.noreply.github.com>

---------

Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com>
Co-authored-by: Eric-Guo <1131536+Eric-Guo@users.noreply.github.com>
Eric-Guo added a commit that referenced this pull request Nov 20, 2025
* Initial plan

* fix: update test snapshots after AGENTS.md refactoring and fix formatting

Co-authored-by: Eric-Guo <1131536+Eric-Guo@users.noreply.github.com>

---------

Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com>
Co-authored-by: Eric-Guo <1131536+Eric-Guo@users.noreply.github.com>
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.

2 participants