chore(charter): absorb ChittyBooks deprecation

[chitcommit]

Summary

• Removes the "ChittyBooks consumes ChittyFinance as engine" line from CHARTER (no longer aspirational — Finance IS the bookkeeping UI after ChittyBooks deprecation)
• Annotates that ledger writes flow through the ChittyLedger substrate as the Finance projection per the canonical Ledger projection model
• Updates "Evidence management" reference to the Evidence projection

Context

ChittyBooks was deprecated in CHITTYAPPS/chittybooks#2 after a 2026-05-27 substance audit found both of its unique features (AI categorization + AI insights) already present in ChittyFinance at higher quality.

Source

CHITTYOS/chittymarket docs/remediation/2026-05-27-finance-books-functioning-as-intended.md

Test plan

• Charter renders cleanly in canon validator
• No code/contract change — docs only

🤖 Generated with Claude Code

Summary by CodeRabbit

• Documentation
  • Updated charter documentation to clarify Finance scope boundaries and explicitly define delegation of Evidence and Legal case management functions to designated systems.

[github-actions]

@coderabbitai review

Please evaluate:

• Security implications
• Credential exposure risk
• Dependency supply chain concerns
• Breaking API changes

[coderabbitai]

📝 Walkthrough

Walkthrough

CHARTER.md is updated to clarify Finance scope boundaries. The "IS NOT Responsible For" section now explicitly delegates evidence management to ChittyLedger's evidence projection and legal case management to ChittyCases. Explanatory text distinguishes ChittyFinance as the bookkeeping UI from ChittyLedger as the system handling ledger writes via the Finance projection.

Changes

Finance Scope Clarification

Layer / File(s)	Summary
Finance scope responsibilities and delegations CHARTER.md	Finance scope's "IS NOT Responsible For" section is revised to clarify that evidence management delegates to ChittyLedger evidence projection, legal case management delegates to ChittyCases, and explanatory text describes ChittyFinance as the bookkeeping UI with ledger writes flowing through ChittyLedger Finance projection.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~3 minutes

Poem

A ledger grows clearer, the boundaries now shine—
Evidence to ChittyLedger, legal to ChittyCases fine!
Finance blooms brighter with delegation well-defined,
Each module knows its role, harmoniously aligned. 🐰✨

🚥 Pre-merge checks | ✅ 5

✅ Passed checks (5 passed)

Check name	Status	Explanation
Description Check	✅ Passed	Check skipped - CodeRabbit’s high-level summary is enabled.
Title check	✅ Passed	The title 'chore(charter): absorb ChittyBooks deprecation' directly and specifically describes the main change: updating CHARTER.md to reflect ChittyBooks deprecation with related annotation updates.
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 unit tests (beta)

• Create PR with unit tests
• Commit unit tests in branch chore/absorb-chittybooks-deprecation

---

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.}

[Copilot]

Pull request overview

Updates the ChittyFinance charter documentation to reflect ChittyBooks’ deprecation and to align the scope language with the Ledger “projection” model used across the ecosystem.

Changes:

• Updates the “IS NOT Responsible For” list to clarify Evidence is handled via ChittyLedger’s Evidence projection.
• Removes the outdated/aspirational ChittyBooks consumer line and replaces it with a deprecation note stating Finance is the bookkeeping UI.
• Adds an explicit note that ledger writes flow through ChittyLedger as the Finance projection (with a canonical URI reference).

---

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

[claude]

Code Review — chore(charter): absorb ChittyBooks deprecation

Scope: Docs-only (CHARTER.md, +9/-2). No code or contract changes.

---

Overview

Removes the stale "ChittyBooks consumes ChittyFinance as engine" line, annotates the Evidence and Finance projections on ChittyLedger, and adds a prose block confirming ChittyFinance is now the bookkeeping UI after the ChittyBooks deprecation (2026-05-27).

---

Issues

1. Structural: prose paragraphs are orphaned inside the IS NOT Responsible For section

The two new paragraphs land after the last bullet of ### IS NOT Responsible For and before ## Dual-Mode Operation, so they visually read as belonging to the IS NOT list — but their content is affirmative ("ChittyFinance IS…", "Ledger writes flow through…"). This is the biggest clarity problem in the diff.

Options:

• Add a ### Notes on Scope (or ### Superseded Services) subheading before these paragraphs to break them out of the IS NOT block.
• Move the ChittyBooks sentence to the Mission section, where affirmative positioning language already lives. The Finance-projection sentence could then live in a brief ### ChittyLedger Integration block — or inline in the Scope list itself (the IS Responsible For list already mentions COA, ledger audit, etc.).

2. Last Updated date not bumped

The footer reads:

*Charter Version: 2.0.0 | Last Updated: 2026-05-02*

This PR is dated 2026-05-27 and modifies the charter. The date should be updated.

3. Canonical URI for Evidence projection not provided (minor)

The Finance projection now has a full URI:

chittycanon://core/services/chittyledger#projection/finance

The Evidence projection only gets a parenthetical label. Consistency would suggest adding its URI too, or intentionally omitting it with a note that it isn't yet registered. Not a blocker.

4. External cross-reference legibility (minor)

See `CHITTYAPPS/chittybooks/ARCHIVED.md`.

This is a path in a different repo. Readers inside this repo have no direct navigation path. A canonical URI (chittycanon://…) or a prose note like "archived in the chittybooks repository" would be more durable than a bare file path that could go stale.

---

What Looks Good

• Removing the ChittyBooks consumer line is the right call — it was aspirational and is now misleading.
• The Finance-projection canonical URI is a useful addition; it ties this doc to the canon model precisely.
• The "Evidence projection" annotation is a small but correct alignment with the projection model.

---

Verdict

The content is correct. The structural placement of the new paragraphs (orphaned between IS NOT and Dual-Mode) and the stale Last Updated date are the two things worth fixing before merge. Everything else is minor polish.

[coderabbitai]

Actionable comments posted: 1

🤖 Prompt for all review comments with AI agents

Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.

Inline comments:
In `@CHARTER.md`:
- Around line 54-60: Update the broken archive link in CHARTER.md by replacing
the incorrect reference `CHITTYAPPS/chittybooks/ARCHIVED.md` with the correct
archived document path (find the actual archived file in the repo or docs site
and use that path), and verify the ledger projection URI
`chittycanon://core/services/chittyledger#projection/finance` is registered in
the canon registry (if missing, add the registration entry where canon services
are declared so the projection exists rather than only being referenced in
CHARTER.md).

🪄 Autofix (Beta)

Fix all unresolved CodeRabbit comments on this PR:

• Push a commit to this branch (recommended)
• Create a new PR with the fixes

---

ℹ️ Review info

⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro

Run ID: a6f4d1f7-99ad-4009-b292-a10ec31163ab

📥 Commits

Reviewing files that changed from the base of the PR and between 652917a and 7f10b84.

📒 Files selected for processing (1)

• CHARTER.md

[coderabbitai]

⚠️ Potential issue | 🟠 Major

🧩 Analysis chain

🏁 Script executed:

#!/bin/bash
# Description: Verify the ChittyBooks archive documentation reference

# Check if the referenced file exists in the chittybooks repository
gh api repos/CHITTYAPPS/chittybooks/contents/ARCHIVED.md --jq '.name' 2>/dev/null || echo "File not found at CHITTYAPPS/chittybooks/ARCHIVED.md"

# Search for any Finance projection canonical URI references in documentation
rg -n "chittycanon://core/services/chittyledger" --type=md -C2

Repository: chittyapps/chittyfinance

Length of output: 490

---

Fix the broken ChittyBooks archive reference in CHARTER.md

• CHITTYAPPS/chittybooks/ARCHIVED.md does not exist at that path (404 from the GitHub contents API); update the reference to the correct archived doc location.
• Ensure chittycanon://core/services/chittyledger#projection/finance is actually registered in canon (right now it’s only referenced in this charter).

🤖 Prompt for AI Agents

Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.

In `@CHARTER.md` around lines 54 - 60, Update the broken archive link in
CHARTER.md by replacing the incorrect reference
`CHITTYAPPS/chittybooks/ARCHIVED.md` with the correct archived document path
(find the actual archived file in the repo or docs site and use that path), and
verify the ledger projection URI
`chittycanon://core/services/chittyledger#projection/finance` is registered in
the canon registry (if missing, add the registration entry where canon services
are declared so the projection exists rather than only being referenced in
CHARTER.md).

[chitcommit]

Closing — depended on now-aborted chittyapps/chittybooks#2 deprecation. ChittyBooks turned out to be under active development (PostgreSQL + ChittyConnect bank-account flow shipped on main during the audit window). Charter line about ChittyBooks consuming Finance is no longer the right edit to make until the two services' boundaries are re-decided by the operator. See chittyapps/chittybooks#2 close comment.