fix(qwen): remove dangling QWEN.md context reference - #40
Open
KDB-Wind wants to merge 1 commit into
Open
Conversation
Remove the contextFileName declaration because the referenced QWEN.md is absent from both the repository and published package. Keep the manifest test focused on the field remaining absent while canonical behavior continues to route through the root skills directory. Document Qwen Code 0.21.1 link and Git-ref install evidence using isolated QWEN_HOME directories without an LLM call. Focused tests and package verification pass; the full-suite result matches the current-main Windows environment baseline.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
Remove the dangling
contextFileName: "QWEN.md"declaration fromqwen-extension.jsonbecause the referenced file does not exist in therepository and is not included in the published npm package.
Update the manifest test to require the field to remain absent. No replacement
QWEN.mdis introduced, so this change does not add extension-wide context oralter the canonical root
skills/behavior.Why
extension manifest.
promise to append extension context from a file the extension cannot provide.
Qwen Code 0.21.1 was verified on Windows through both registration paths with
isolated
QWEN_HOMEdirectories:qwen extensions link <local-path>qwen extensions install <git-repository-url>Current upstream displayed the same dangling
QWEN.mdcontext message throughboth paths. The fixed branch omitted the message while retaining the
better-harnessSkill. A fixed Git-ref install was completed inside theisolated home and confirmed with
qwen extensions list; no chat or LLM callwas made.
Traceability and Scope
docs/specs/2026-07-29-qwen-extension-context-file.mdqwen-extension.jsonno longer declarescontextFileName; allremaining manifest fields are unchanged.
test/plugin-manifests.test.mjsrequiresqwen.contextFileName === undefined.version-aligned with
package.json.promise, retain Skill discovery, and can be listed from an isolated home.
qwen-extension.jsontest/plugin-manifests.test.mjsdocs/specs/2026-07-29-qwen-extension-context-file.mdQWEN.md.QWEN.mdinventory.Change Type
Test and Review Evidence
node --test test/plugin-manifests.test.mjs test/doc-link-graph.test.mjsnpm run pack:verifygit diff --check origin/mainnpm testnpm teston unmodified current upstreamManual evidence: the repository contains no root
QWEN.md, and thepackage.jsonfileslist does not ship one. The staged patch was alsocompared with the reviewed source patch before the final commit was created.
The two full-suite failures reproduce on unmodified current upstream:
CLAUDE.md;simulated command exits zero in this environment.
Focused affected-area tests and package verification pass. Clean GitHub CI
remains the authoritative full-suite result.
Risk and Recovery
platform-neutral JSON. It removes an undeliverable context reference and
leaves Skill discovery unchanged.
qwen-extension.jsonchanges by one optional field. Package verificationpasses; no schema or generated file changes.
extension-owned
QWEN.mdinstead. That would be a product decision requiringthe file, package inclusion, context content review, and an existence test.
declaration for behavior that was never deliverable; it introduces no new
capability.
AI Involvement
patch, spec, and validation workflow. The contributor made and reviewed the
remove-versus-add product decision, inspected the final diff and acceptance
criteria, and checked the recorded validation evidence. The focused tests,
doc-link checks, package verification, and diff checks were rerun after the
final commit was rebuilt.
Checklist
AGENTS.md,CONTRIBUTING.md, and the relevant canonical-owner guidance.CHANGELOG.md, or the no-entry rationale is documented above.