Skip to content

Fablize integration#515

Closed
k6bdptd77n-arch wants to merge 3 commits into
DeusData:mainfrom
k6bdptd77n-arch:fablize-integration
Closed

Fablize integration#515
k6bdptd77n-arch wants to merge 3 commits into
DeusData:mainfrom
k6bdptd77n-arch:fablize-integration

Conversation

@k6bdptd77n-arch

Copy link
Copy Markdown

What does this PR do?

Checklist

  • Every commit is signed off (git commit -s) — required, CI rejects
    unsigned commits (DCO, see CONTRIBUTING.md)
  • Tests pass locally (make -f Makefile.cbm test)
  • Lint passes (make -f Makefile.cbm lint-ci)
  • New behavior is covered by a test (reproduce-first for bug fixes)

ilkhombek4saves-sudo and others added 3 commits June 18, 2026 17:34
Pairs the codebase-memory-mcp engine (what the code IS) with fablize
(how to work on it): clarify-first, multi-story verification gate with
bounded self-correction, investigation protocol, verification grounding,
destructive-action guard, and observability.

Integration is prompt-level only — the disciplines call the MCP tools
(get_architecture, search_graph, trace_path, detect_changes) at the points
they help most (see INTEGRATION.md). The C core is unmodified upstream, so
git pull upstream merges cleanly; fablize lives entirely in fablize/ as
dependency-free stdlib Python + plain-text packs.

- fablize/: packs, engines (goals/spec/metrics/bundle), guard hook, 15 tests
- install-combined.sh: build engine -> register MCP -> apply disciplines
- INTEGRATION.md, NOTICE (dual-MIT), fablize-ci.yml (path-scoped)
- README: additive callout for the combined distribution

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
Makes the Claude Code economy/memory layer and Hermes economizer
self-contained inside the project so the external imba/ folder can be removed.
Run imba/setup.sh to (re)install on a new machine.

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
@DeusData

Copy link
Copy Markdown
Owner

Hey, the CI is red. In case you are interested for a merge, please fix the issues. Otherwise thanks for the contribution :) Will only review PRs which pass the CI checks

@DeusData

Copy link
Copy Markdown
Owner

Thanks for the submission, but we can't take this PR.

It's well outside what we can review or accept: it bundles multiple unrelated projects together in one change (~3,000 lines across 50 files), adds a new CI workflow, and ships install scripts that modify global user configuration and agent-instruction files. That's not a change we can vet or maintain, and it doesn't match the stated title.

If there's a specific, self-contained improvement you'd like to propose, please open an issue first so we can agree on scope — then a focused PR limited to that one change, with a complete description, DCO sign-off, no bundled second project, and no installer that writes to user configuration.

Closing for now.

@DeusData DeusData closed this Jun 22, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants