fix: issue 104

[muratkeremozcan]

Summary

• Clarifies that workflow-local TEA knowledge resources are intentional self-contained skill packaging.
• Adds install validation that workflow knowledgeIndex entries resolve to local indexes and fragment files.

Addresses #104.

Validation

• npm run test:install
• npm run test:knowledge
• npm run lint:md
• npm run format:check
• npm run lint
• npm run docs:validate-links

[augmentcode]

🤖 Augment PR Summary

Summary: This PR addresses issue #104 by clarifying and enforcing how knowledge resources are packaged for TEA and workflow skills.

Changes:

• Documented that workflow skills intentionally duplicate knowledge resources to remain self-contained and modular
• Updated knowledge-base documentation to explain agent-level vs workflow-local indexes and path resolution via {skill-root}
• Extended installation component tests to validate workflow-local resources/tea-index.csv usage and that indexed fragments exist
• Added CSV parsing to validate workflow knowledge index records against local fragment files

_{🤖 Was this summary useful? React with 👍 or 👎}

[augmentcode]

Review completed. 1 suggestion posted.

Comment augment review to trigger a new review at any time.

[coderabbitai]

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info

⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro

Run ID: 3ccf8b30-43ec-4974-9c17-45edfd71f223

📥 Commits

Reviewing files that changed from the base of the PR and between 4360295 and 665facf.

📒 Files selected for processing (4)

• CHANGELOG.md
• README.md
• docs/explanation/knowledge-base-system.md
• test/test-installation-components.js

---

📝 Walkthrough

Walkthrough

This PR documents workflow-local knowledge resources as intentional self-contained skill packaging and adds validation to ensure workflow knowledgeIndex references resolve to parseable CSV files with matching fragment files on disk. All ranges are assigned to coherent developer checkpoints covering documentation and test validation.

Changes

Knowledge Resource Documentation and Validation

Layer / File(s)	Summary
Knowledge resource architecture documentation CHANGELOG.md, README.md, docs/explanation/knowledge-base-system.md	Changelog entry, README Building Blocks table, and knowledge-base-system documentation all clarify that workflow-local knowledge resources (resources/tea-index.csv and resources/knowledge/*.md) are intentionally duplicated for self-contained skill packaging, with path resolution via frontmatter {skill-root}.
Knowledge index validation in test suite test/test-installation-components.js	Test file imports CSV parser, introduces per-workflow validation guard, and extends step validation to confirm knowledgeIndex references exist and are parseable, verify all CSV records have valid fragment_file fields pointing to existing files, and assert record count matches fragment file count.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 minutes

Possibly related PRs

• bmad-code-org/bmad-method-test-architecture-enterprise#63: Both PRs modify the knowledge-base/tea-index.csv path handling in test/test-installation-components.js, with the main PR extending it to validate workflow knowledgeIndex CSV/fragment references while PR #63 shifts the baseline tea-index.csv/fragments location to the new src/agents/bmad-tea/resources layout.

🚥 Pre-merge checks | ✅ 3 | ❌ 2

❌ Failed checks (1 warning, 1 inconclusive)

Check name	Status	Explanation	Resolution
Description check	⚠️ Warning	No pull request description was provided by the author, making it impossible to assess whether it relates to the changeset.	Add a description explaining the purpose of the changes, such as documenting workflow-local knowledge resources and adding validation for knowledge indexes.
Title check	❓ Inconclusive	The title 'fix: issue 104' is vague and does not convey meaningful information about the changeset; it lacks descriptive content about what was actually fixed.	Replace with a descriptive title that summarizes the main change, such as 'Add validation for workflow-local knowledge resource indexes'.

✅ Passed checks (3 passed)

Check name	Status	Explanation
Docstring Coverage	✅ Passed	No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check	✅ Passed	Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check	✅ Passed	Check skipped because no linked issues were found for this pull request.

_{✏️ Tip: You can configure your own custom pre-merge checks in the settings.}

✨ Finishing Touches

📝 Generate docstrings

• Create stacked PR
• Commit on current branch

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Commit unit tests in branch fix/issue-104

Tip

💬 Introducing Slack Agent: The best way for teams to turn conversations into code.

Slack Agent is built on CodeRabbit's deep understanding of your code, so your team can collaborate across the entire SDLC without losing context.

• Generate code and open pull requests
• Plan features and break down work
• Investigate incidents and troubleshoot customer tickets together
• Automate recurring tasks and respond to alerts with triggers
• Summarize progress and report instantly

Built for teams:

• Shared memory across your entire org—no repeating context
• Per-thread sandboxes to safely plan and execute work
• Governance built-in—scoped access, auditability, and budget controls

One agent for your entire SDLC. Right inside Slack.

👉 Get started

---

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands and usage tips.}