From cf1b444a452896de374dacaa23b4fcb654c28f77 Mon Sep 17 00:00:00 2001
From: reachableceo <charles@turnsys.com>
Date: Mon, 10 Aug 2026 09:49:21 -0500
Subject: [PATCH] refactor(rules): point discourse-cli invocation at
 centralized creds store
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

Merge discourse-cli into tooling-cli/discourse and drop the
discourse-cli/.env source path from the rules. The canonical
invocation now uses ~/.creds/discourse.env (system-independent)
with the registry image. Marks Q2 (bin/ wrapper gap) resolved.

💘 Generated with Crush

Assisted-by: Crush:glm-5.2
---
 AGENTS.md       |  4 ++--
 questions-v1.md | 14 ++++++--------
 2 files changed, 8 insertions(+), 10 deletions(-)

diff --git a/AGENTS.md b/AGENTS.md
index 88b6653..2140469 100644
--- a/AGENTS.md
+++ b/AGENTS.md
@@ -71,8 +71,8 @@ docker run --rm --env-file ~/projects/KNEL-AIMiddleware/redmine-cli/.env \
     git.knownelement.com/reachableceo/redmine-cli:latest <command>
 # subcommands: whoami, projects, statuses, list, show <id>, create, update <id>, close <id>
 
-# discourse-cli  (source: ~/projects/KNEL-AIMiddleware/discourse-cli/)
-docker run --rm --env-file ~/projects/KNEL-AIMiddleware/discourse-cli/.env \
+# discourse-cli
+docker run --rm --env-file ~/.creds/discourse.env \
     git.knownelement.com/reachableceo/discourse-cli:latest <command>
 # subcommands: whoami, categories, cat-info, topics, show <id>, create, reply <id>, update <post_id>, delete <post_id>, search
 ```
diff --git a/questions-v1.md b/questions-v1.md
index 2248587..eec4655 100644
--- a/questions-v1.md
+++ b/questions-v1.md
@@ -16,14 +16,12 @@
 
 Go with option a. The tea command is setup on this workstation (and on ultix-offstage). I guess, also capture that the tea command (and docker login) are setup on my workstations, so that in the future, projects know they can use tea to setup a repo. Also, i want this to be TSYS wide, so it should go under the TSYSGroupCorporate organization. Call the repo: TSYSGroupAIOS . Make it a template repository.
 
-### Q2. The bin/ wrapper gap (redmine-cli / discourse-cli)
-- **Context:** PFVCluster's operational.md and AGENTS.md reference `~/daytoday/redmine/bin/redmine` and `~/daytoday/discourse/bin/discourse` as the entrypoints. But `ls ~/daytoday/{redmine,discourse}/` shows only `.gitignore` + `AGENTS.md` (+ MIGRATION-PLAN.md for discourse) — no `bin/`, no Dockerfile. The actual CLI source lives in `~/projects/KNEL-AIMiddleware/{redmine,discourse}-cli/`.
-- **Question:** Are the `bin/` wrappers something that should exist (and were lost / never committed), or is the documentation aspirational? Should the template reference these CLIs at all, or stay tool-agnostic?
-- **Answer:** _(human)_
-- **Decision:** _(human/agent)_
-- **Synthesized to:** —
-
-The clis should exist. Maybe the AGENTS.md reference the actual paths? I dont need duplicate code. I think i was using the directories as kind of "shortcuts" vs the ~/projects/... path every time. So, for this repo, reference the full path/container name/invoke notes. Does that make sense?
+### Q2. The bin/ wrapper gap (redmine-cli / discourse-cli) — RESOLVED (discourse)
+- **Context:** Projects referenced `~/daytoday/redmine/bin/redmine` and `~/daytoday/discourse/bin/discourse` as entrypoints — thin shortcut wrappers around the real CLI containers. The actual CLI source lived in `~/projects/KNEL-AIMiddleware/{redmine,discourse}-cli/`.
+- **Question:** Are the `bin/` wrappers something that should exist, or is the documentation aspirational? Should the template reference these CLIs at all, or stay tool-agnostic?
+- **Answer:** Reference the real container invocation (full path/container name/invoke notes); no duplicate code via shortcut wrappers.
+- **Decision:** No `bin/` wrappers. Invoke the real container with `docker run`. The discourse-cli source has been merged into `~/projects/KNEL-AIMiddleware/tooling-cli/discourse/` (CLI source + Dockerfile + README + AGENTS.md + validate.sh). The canonical invocation is documented in AGENTS.md §CLI invocation and uses `--env-file ~/.creds/discourse.env` (centralized credential store) — no system-dependent paths in the rules. The old `~/daytoday/discourse` workspace and `discourse-cli/` subdir were removed. Redmine still pending the same treatment.
+- **Synthesized to:** `~/daytoday/meta/AGENTS.md` §CLI invocation; `tooling-cli/discourse/`
 
 ### Q3. Should the template ship the Discourse pointer-header pattern?
 - **Context:** PFVCluster migrated 36 in-repo `.md` files to 10-line pointer stubs citing `https://community.turnsys.com/t/<N>`. The template currently has `scripts/garden.sh` that *warns* about oversized non-Discourse `.md`, but doesn't enforce the pointer-header format.