8000 feat(notes): count note refinement and filing in the activity ledger by adeelahmad · Pull Request #133 · adeelahmad/lense · GitHub
[go: up one dir, main page]

Skip to content

feat(notes): count note refinement and filing in the activity ledger - #133

Merged
adeelahmad merged 1 commit into
mainfrom
claude/notes-module-j9j30y
Oct 4, 2026
Merged

adeelahmad merged 1 commit into
mainfrom
claude/notes-module-j9j30y

Conversation

@adeelahmad
Copy link
Copy Markdown
Owner

Description

Before: the model calls that refine a note's title and summary, and the decisions that file it, weren't counted against the note in the activity ledger. Requests to /api/v1/notes/{id} were counted as note:{id}, which is a recording's notes table.

After: those calls count for the note (note_page:5) and its namespace. Requests to note pages count for the note page, and a note can be made with POST /api/v1/notes. Anyone who can read the namespace can see a note's history, and each note page ends with "Costs and activity".

Motivation and Context

Notes module: "keep associating cost", through the activity ledger from #114. The /api/v1/notes path mapping is a small fix in activity._table, shared by path refs and POST replies. A recording's notes/{id} keeps meaning its notes.

Types of changes

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to change; ! in the title)

Checklist:

  • The title is a Conventional Commit.
  • My code follows the code style of this project (make hooks checks it on commit).
  • I added or updated tests.
  • My change requires documentation updates, and I have updated docs/ accordingly.
  • Users need more than the title to understand it, so I wrote it under ## Unreleased in CHANGELOG.md.

Checked locally: the full backend suite passed (597, with the known LibreOffice conversion test deselected), along with ruff, tsc and eslint. A second run on top of #119 is in progress together with the next notes change.

🤖 Generated with Claude Code

https://claude.ai/code/session_01XZQXJtCvJgbG1Z2H312KcS


Generated by Claude Code

Model calls that refine a note's title and summary, and the decisions
that file it in PARA, count for the note (note_page:id) and its
namespace. Requests to /api/v1/notes count for the note page rather
than a recording's notes, readers of the namespace can see a note's
history, and each note shows its costs and activity.

Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01XZQXJtCvJgbG1Z2H312KcS
@adeelahmad
adeelahmad force-pushed the claude/notes-module-j9j30y branch from 12d9648 to 087fb44 Compare October 4, 2026 13:08
@adeelahmad
adeelahmad merged commit 5c7c8aa into main Oct 4, 2026
0 of 5 checks passed
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.

2 participants

0