Skip to content
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
26 changes: 26 additions & 0 deletions memory-bank/research/R-GH-87/README.md
Original file line number Diff line number Diff line change
@@ -0,0 +1,26 @@
---
title: "R-GH-87: Validation Tool Ownership"
doc_kind: research
doc_function: index
purpose: "Navigation for the evidence-backed decision package R-GH-87."
derived_from:
- ../../flows/research.md
- brief.md
status: active
audience: humans_and_agents
---

# R-GH-87: Validation Tool Ownership

## Lifecycle Owner

The current lifecycle state is owned only by the `research_status` field in
[Research Brief](brief.md).

## Annotated Index

- [Research Brief](brief.md) — canonical decision question, boundaries,
assumptions and stopping condition.
- [Evidence Log](evidence.md) — source provenance and observations.
- [Research Synthesis](synthesis.md) — findings, confidence and limitations.
- [Research Decision](decision.md) — decision rationale and handoff map.
86 changes: 86 additions & 0 deletions memory-bank/research/R-GH-87/brief.md
Original file line number Diff line number Diff line change
@@ -0,0 +1,86 @@
---
title: "R-GH-87: Validation Tool Ownership"
doc_kind: research
doc_function: canonical
purpose: "Canonical decision question, boundaries and lifecycle state for R-GH-87."
derived_from:
- ../../flows/research.md
status: active
research_status: validated
audience: humans_and_agents
---

# R-GH-87: Validation Tool Ownership

## Intake

| Field | Value |
| --- | --- |
| Source / trigger | [dapi/memory-bank#87](https://github.com/dapi/memory-bank/issues/87) |
| Research owner | delivery orchestrator |
| Decision owner | `dapi/memory-bank` maintainer |
| Research mode | `technical_discovery` |
| Decision deadline / timebox | Bounded desk research completed 2026-08-05 |

## Decision Question

- `RQ-01` Does the current repository still contain validation implementation
that should be removed or moved to `dapi/memory-bank-cli`, and is a new
standalone CLI task required?

## Working Hypotheses

- `HYP-01` Reusable `lint` and `doctor` validation is already owned by
`dapi/memory-bank-cli`; the remaining Ruby checks in this repository are
producer-side integration checks and should remain here.

## Compact Method Record

- Method and source strategy: inspect the issue and maintainer routing record,
the migration commit, the current CI workflow and the linked CLI migration
issue.
- Collection window and context: GitHub and the checked-out repository on
2026-08-05; current `main` at `23ff0a3905645c6f4e37809ae7e97d43cb937a37`.
- Evidence-quality criteria: primary GitHub issue/commit/workflow sources;
distinguish current executable use from historical intent.
- Applicable privacy, consent, legal, security and vendor-access constraints:
none; only public repository metadata and checked-out source are used.
- Bias risk and disconfirming signal: prior maintainer conclusions could be
stale; current CI still invoking local reusable lint/doctor implementation
would contradict `HYP-01`.

## Scope

- `RSC-01` Ownership of validation used by this repository's CI and remaining
`tools/` scripts, relative to `memory-bank-cli`.

## Non-Scope

- `RNS-01` Changing CLI behavior, release process, or template CI contracts.
- `RNS-02` Deleting repository-specific checks without a separately routed
delivery task.

## Assumptions and Known Evidence

| ID | Statement | Type | Source / confidence |
| --- | --- | --- |
| `ASM-01` | The issue seeks a decision before any deletion or new CLI work. | Evidence | [Issue #87](https://github.com/dapi/memory-bank/issues/87) |
| `ASM-02` | The maintainer is the decision owner for this repository decision. | Evidence | [Routing comment](https://github.com/dapi/memory-bank/issues/87#issuecomment-5087962973) |

## Stopping Condition

- `STOP-01` Stop once the migration history, current CI ownership boundary and
linked CLI migration task either support or refute `HYP-01`.

## Open Questions

| Question | Blocks | Owner | Resolution evidence |
| --- | --- | --- | --- |
| None | — | — | `STOP-01` is satisfied. |

## Boundary Check

- [x] This brief contains a question and hypothesis, not findings presented as facts.
- [x] Every known fact has a clickable source link.
- [x] No committed delivery scope, selected solution, ADR decision or implementation sequence is defined here.
- [x] Required privacy, consent, legal, security and access constraints are explicitly `none`.
62 changes: 62 additions & 0 deletions memory-bank/research/R-GH-87/decision.md
Original file line number Diff line number Diff line change
@@ -0,0 +1,62 @@
---
title: "R-GH-87: Research Decision"
doc_kind: research
doc_function: canonical
purpose: "Decision rationale and promotion map for R-GH-87."
derived_from:
- brief.md
- synthesis.md
- ../../flows/research.md
status: active
audience: humans_and_agents
---

# R-GH-87: Research Decision

## Decision

| Field | Value |
| --- | --- |
| Decision owner | `dapi/memory-bank` maintainer |
| Decision date | 2026-08-05 |
| Decision reference | [Issue #87 validated decision comment](https://github.com/dapi/memory-bank/issues/87#issuecomment-5187546909) and this evidence package |

Terminal disposition is recorded only in sibling `brief.md` as
`research_status: validated`.

## Decision Rationale

- `FND-01` and `FND-02` show a clean ownership boundary: reusable validation
is in the standalone CLI, while the remaining Ruby scripts are local
producer-side integration validation.
- `FND-03` confirms that the task which performed the CLI migration is closed;
no new reusable CLI requirement is evidenced here.
- `LIM-01` means this is not an assertion that the Ruby checks must remain
forever; it rejects deletion or a duplicate CLI task on the present evidence.

## Recommendation

- `REC-01` Retain `tools/validate-priming-manifests*.rb`, create no duplicate
`memory-bank-cli` issue, and record the ownership decision in this completed
research package. Confidence: high for ownership; see `LIM-01` for the
boundary of the conclusion.

## Alternatives Considered

| Alternative | Why not selected / what would change the decision |
| --- | --- |
| Delete remaining `tools/` scripts now. | Rejected: current CI invokes them as a distinct producer-side integration surface; a separately routed removal/change would need evidence that this coverage is obsolete. |
| Create a new CLI issue. | Rejected: CLI migration is already closed and this research identifies no missing reusable CLI contract. A concrete reusable requirement would justify new Task Routing. |

## Promotion and Handoff Map

| ID | Accepted or retained fact | Canonical downstream owner | Target route / link |
| --- | --- | --- | --- |
| `HD-01` | The current Ruby priming-manifest checks remain repository-specific integration validation. | This package's retained decision record; no new active project fact is required. | No follow-up delivery route. |
| `HD-02` | Future extraction requires a concrete reusable CLI contract. | A new issue owned by the requester. | New [Task Routing](../../flows/routing.md) only if that requirement appears. |

## Closure Check

- [x] Sibling `brief.md` records matching terminal `research_status: validated`.
- [x] `synthesis.md` answers `RQ-01`, and this recommendation traces to `FND-*` and `LIM-*`.
- [x] Handoff creates neither delivery scope nor implementation steps by implication.
47 changes: 47 additions & 0 deletions memory-bank/research/R-GH-87/evidence.md
Original file line number Diff line number Diff line change
@@ -0,0 +1,47 @@
---
title: "R-GH-87: Evidence Log"
doc_kind: research
doc_function: canonical
purpose: "Traceable evidence and observations collected for R-GH-87."
derived_from:
- brief.md
- ../../flows/research.md
status: active
audience: humans_and_agents
---

# R-GH-87: Evidence Log

## Sources

| ID | Source / provenance | Date / freshness | Collection context | Access / quality note |
| --- | --- | --- | --- | --- |
| `SRC-01` | [Issue #87](https://github.com/dapi/memory-bank/issues/87) | 2026-08-05 | GitHub issue read | Primary task source; wording is underspecified. |
| `SRC-02` | [Maintainer routing record](https://github.com/dapi/memory-bank/issues/87#issuecomment-5087962973) | 2026-07-27 | GitHub issue comment | Primary decision-owner record; verify against current sources. |
| `SRC-03` | [Migration commit `5d1d347`](https://github.com/dapi/memory-bank/commit/5d1d3473e1473ccad19f2b8a2848fe92639f1887) | 2026-07-23 | Git history inspection | Primary change history; proves files changed, not runtime behavior. |
| `SRC-04` | [Current CI workflow](https://github.com/dapi/memory-bank/blob/23ff0a3905645c6f4e37809ae7e97d43cb937a37/.github/workflows/ci.yml) | 2026-08-05 | Checked-out `main` and GitHub source | Primary executable CI configuration. |
| `SRC-05` | [memory-bank-cli#1](https://github.com/dapi/memory-bank-cli/issues/1) | closed 2026-07-22 | GitHub issue read | Primary migration-task record; its acceptance explicitly moved the CLI and removed old executable references. |
| `SRC-06` | [Successful CI run 30912292219](https://github.com/dapi/memory-bank/actions/runs/30912292219) | 2026-08-05 | GitHub Actions metadata | Current integration result; does not alone prove semantic coverage. |

## Observations

| ID | Observation | Supporting `SRC-*` | Applies to | Interpretation boundary |
| --- | --- | --- | --- | --- |
| `OBS-01` | Commit `5d1d347` removed the Go CLI under `tools/`, including both previous executable entry points and lint/doctor implementation, while updating CI and documentation. | [SRC-03](#sources) | `RQ-01`, `HYP-01` | Does not prove no repository-local scripts remain. |
| `OBS-02` | Current CI installs `memory-bank-cli` for `lint` and `doctor`, but runs `ruby tools/validate-priming-manifests*.rb` separately before those CLI commands. | [SRC-04](#sources) | `RQ-01`, `HYP-01` | Shows execution ownership, not whether the Ruby checks are useful. |
| `OBS-03` | The linked CLI task is closed and defined the move to a standalone `memory-bank-cli` with preserved `lint` and `doctor` semantics. | [SRC-05](#sources) | `RQ-01`, `HYP-01` | Does not establish a new CLI feature is impossible. |
| `OBS-04` | Current `main` CI completed successfully for the examined integration workflow. | [SRC-06](#sources) | `RQ-01` | A passing run does not independently establish completeness. |

## Collection Log

| Date | Activity | Result | Deviation / reason |
| --- | --- | --- | --- |
| 2026-08-05 | Read issue, prior maintainer record and linked CLI issue. | Decision context and migration acceptance recovered. | No external participant or privileged source was needed. |
| 2026-08-05 | Inspected migration commit and current CI workflow. | Reusable CLI and repository-specific script surfaces distinguished. | None. |

## Evidence Quality Check

- [x] Each material observation traces to one or more `SRC-*`.
- [x] Every `SRC-*` contains a clickable original-source link.
- [x] Observations are separated from source claims and analyst interpretation.
- [x] Freshness, source limitations and conflicts are recorded.
42 changes: 42 additions & 0 deletions memory-bank/research/R-GH-87/synthesis.md
Original file line number Diff line number Diff line change
@@ -0,0 +1,42 @@
---
title: "R-GH-87: Research Synthesis"
doc_kind: research
doc_function: canonical
purpose: "Findings, confidence and limitations synthesized from evidence for R-GH-87."
derived_from:
- brief.md
- evidence.md
status: active
audience: humans_and_agents
---

# R-GH-87: Research Synthesis

## Findings

| ID | Finding | Evidence | Confidence | Implication for `RQ-*` / `HYP-*` |
| --- | --- | --- | --- | --- |
| `FND-01` | Reusable `lint` and `doctor` implementation was moved out of this repository into `memory-bank-cli`. | [OBS-01](evidence.md#observations), [OBS-03](evidence.md#observations) | High | Supports `HYP-01`; no local Go CLI remains to remove. |
| `FND-02` | Remaining Ruby priming-manifest checks are an explicit repository CI surface run alongside, not through, the installed CLI. | [OBS-02](evidence.md#observations) | High | Supports retaining the checks under this repository's ownership. |
| `FND-03` | The migration task that established the standalone CLI is already completed, and current main CI is green. | [OBS-03](evidence.md#observations), [OBS-04](evidence.md#observations) | Medium | Does not justify a duplicate CLI issue without a new CLI requirement. |

## Limitations and Disconfirming Evidence

| ID | Limitation / conflicting signal | Effect on conclusion | Mitigation or next question |
| --- | --- | --- | --- |
| `LIM-01` | This is a bounded source review, not a semantic redesign of the Ruby validation. | The recommendation does not claim the scripts can never move. | Route a new technical discovery or delivery issue only if a concrete reusable CLI contract is proposed. |
| `LIM-02` | Green CI proves execution on current main, not every possible regression case. | Confidence in ownership is unaffected; coverage adequacy remains outside this issue. | Evaluate coverage independently if a future issue identifies a gap. |

## Answer to Decision Question

No additional removal is supported. The reusable validation requested by the
issue is already in `dapi/memory-bank-cli`; the remaining Ruby checks validate
this repository's priming-manifest producer and CI integration. A new CLI task
would duplicate completed migration work unless a separate requirement defines
a reusable CLI contract for those checks.

## Review Check

- [x] Every finding traces through linked observations to linked sources.
- [x] Confidence reflects source quality rather than the desired outcome.
- [x] Alternative explanation and residual uncertainty are visible.
3 changes: 3 additions & 0 deletions memory-bank/research/README.md
Original file line number Diff line number Diff line change
Expand Up @@ -31,3 +31,6 @@ audience: humans_and_agents
## Instantiated Research

В шаблонном репозитории этот каталог может быть пустым. Это нормально.

- [`R-GH-87/`](R-GH-87/README.md) — decision record for ownership of reusable
CLI validation and repository-specific priming-manifest checks.
Loading