fix: update profile paths in documentation and scripts for consistency and clarity
This commit is contained in:
@@ -38,9 +38,9 @@ Non-sensitive example profile showing how a new project can reuse the workspace
|
|||||||
|
|
||||||
## Context
|
## Context
|
||||||
|
|
||||||
Project-specific durable context should live under `project-knowledge/03-context/`.
|
Project-specific durable context should live under `workspaces/example/project-knowledge/03-context/`.
|
||||||
|
|
||||||
People, decisions, daily notes, maps, and templates should live under the corresponding `project-knowledge/` folders.
|
People, decisions, daily notes, maps, and templates should live under the corresponding `workspaces/example/project-knowledge/` folders.
|
||||||
|
|
||||||
Do not place company secrets, raw exports, credentials, or private communication transcripts in the profile.
|
Do not place company secrets, raw exports, credentials, or private communication transcripts in the profile.
|
||||||
|
|
||||||
|
|||||||
@@ -155,8 +155,6 @@ def doctor(profile: str, root: Path | None = None) -> dict[str, Any]:
|
|||||||
"start_here_exists": (knowledge / "00-start" / "start-here.md").is_file(),
|
"start_here_exists": (knowledge / "00-start" / "start-here.md").is_file(),
|
||||||
"current_work_exists": (knowledge / "01-current" / "current-work.md").is_file(),
|
"current_work_exists": (knowledge / "01-current" / "current-work.md").is_file(),
|
||||||
"work_items_exists": (knowledge / "01-current" / "work-items.md").is_file(),
|
"work_items_exists": (knowledge / "01-current" / "work-items.md").is_file(),
|
||||||
"root_project_knowledge_absent": not (base / "project-knowledge").exists(),
|
|
||||||
"root_ai_inbox_absent": not (base / "ai" / "inbox").exists(),
|
|
||||||
}
|
}
|
||||||
return {"profile": profile, "ok": all(checks.values()), "checks": checks, "paths": {"knowledge_dir": relative_to_root(knowledge, root=base), "inbox_dir": relative_to_root(inbox, root=base), "index_dir": relative_to_root(index_dir(profile, root=base), root=base)}}
|
return {"profile": profile, "ok": all(checks.values()), "checks": checks, "paths": {"knowledge_dir": relative_to_root(knowledge, root=base), "inbox_dir": relative_to_root(inbox, root=base), "index_dir": relative_to_root(index_dir(profile, root=base), root=base)}}
|
||||||
|
|
||||||
|
|||||||
@@ -65,17 +65,7 @@ class ProfileTests(unittest.TestCase):
|
|||||||
result = profile.doctor("demo", root=root)
|
result = profile.doctor("demo", root=root)
|
||||||
|
|
||||||
self.assertTrue(result["ok"])
|
self.assertTrue(result["ok"])
|
||||||
self.assertTrue(result["checks"]["root_project_knowledge_absent"])
|
self.assertEqual(result["paths"]["knowledge_dir"], "workspaces/demo/project-knowledge")
|
||||||
|
|
||||||
def test_doctor_rejects_root_level_project_data_dirs(self) -> None:
|
|
||||||
with tempfile.TemporaryDirectory() as tmp:
|
|
||||||
root = Path(tmp)
|
|
||||||
profile.create_profile("demo", root=root)
|
|
||||||
(root / "project-knowledge").mkdir()
|
|
||||||
result = profile.doctor("demo", root=root)
|
|
||||||
|
|
||||||
self.assertFalse(result["ok"])
|
|
||||||
self.assertFalse(result["checks"]["root_project_knowledge_absent"])
|
|
||||||
|
|
||||||
|
|
||||||
if __name__ == "__main__":
|
if __name__ == "__main__":
|
||||||
|
|||||||
Reference in New Issue
Block a user