SCALE-SEAM (99) — the content shelf, and a destination header that was wrong until now

[ibuilder]

What & why

apps/web/src/api/client.ts goes 665 → 648 (62 methods above the STAYING banner, 4 below).
contentCatalog, placeContent and importContent move to the existing
apps/web/src/api/authoring.ts. What they answer: what pre-made content can I place, and place
it? Continues SCALE-SEAM in docs/roadmap.md.

A role-for-role parallel, read off the signatures

The family shelf was already in that file. The content shelf is the same three roles, with matching
shapes and arities:

role	family	content
catalog reader	familyCatalog() → {count, categories: Record<…>}	contentCatalog() → {count, note, groups: Record<…>}
placer	placeFamily(pid, family, position)	placeContent(pid, category, point, name)
multipart importer	async importFamilies(pid, file, …)	async importContent(pid, file, opts)

A parallel between two method triples is structural. "Both are shelves" would have been a
shared word — the grouping (88) and (89) each had to reject — so the argument here is deliberately
the signatures, not the noun.

The destination's own first line was wrong until this commit

authoring.ts line 1 has read "the family/content shelf" while the file held zero content
methods; the word's only other appearances there are an HTTP header and a sentence about IFC type
content. The docstring stated an intended scope as though it were a fact.

This is recorded as corroboration that was false, not as evidence. A header agreeing with the
answer is worth nothing until someone checks whether it is true — and this one had been wrong for as
long as it had existed. It is the smallest possible instance of the drift this repo's instructions
keep warning about, and it is exactly the kind of agreement that is tempting to cite instead of
verify.

Note on this branch's history

#431 was squash-merged, so origin/main carries (98) as ae4bc0e7 while this branch still held the
pre-squash commits. The content diff between the remote branch and origin/main was verified empty
first, so the branch held no unmerged work; the histories were reconciled with a -s ours merge
rather than a force-push.

Checklist (mirrors CONTRIBUTING.md)

• Backend: cd services/api && python -m ruff check ../.. clean
• Backend: test_file_sizes.py, test_claude_md_gates.py, test_roadmap_status.py exit 0;
  full suite running as a cross-check
• Web: npx tsc --noEmit 0, npm run lint 0, npm run build 0;
  npx vitest run src/api 27 files / 119 tests
• No import cycles introduced
• CHANGELOG.md entry added (newest at top); docs/roadmap.md updated
• Version bump — n/a, not a release PR
• No secrets, no competitor names in shipped docs

🤖 Generated with Claude Code

https://claude.ai/code/session_01Tt2XKB83wwNt2nrMbK6eEA

---

Generated by Claude Code

Summary by CodeRabbit

• API Improvements

  • Content catalog browsing, content placement, and multipart content importing are now available through the authoring API.
  • Content placement supports location, category, naming, and publishing options.
  • Content importing supports category, scale, storey, naming, and related import options.

• Documentation

  • Updated API and roadmap documentation to reflect the available content-authoring capabilities.

[coderabbitai]

Caution

Review failed

The pull request is closed.

ℹ️ Recent review info

⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Team

Run ID: 7a1d3079-1967-483d-823e-41861159ab7d

📥 Commits

Reviewing files that changed from the base of the PR and between ae4bc0e and 17103db.

📒 Files selected for processing (6)

• CHANGELOG.md
• apps/web/src/api/authoring.ts
• apps/web/src/api/client.ts
• apps/web/src/api/surface.test.ts
• docs/roadmap.md
• services/api/test_file_sizes.py

---

📝 Walkthrough

Walkthrough

This change moves three content authoring methods from ApiClient to withAuthoring, preserves their behavior, updates API surface checks, and records the extraction in project tracking and file-size limits.

Changes

Content shelf extraction

Layer / File(s)	Summary
Authoring client methods apps/web/src/api/authoring.ts	Adds contentCatalog, placeContent, and importContent with catalog retrieval, placement, multipart upload, query options, error handling, and typed responses.
API surface and tracking updates apps/web/src/api/client.ts, apps/web/src/api/surface.test.ts, services/api/test_file_sizes.py, CHANGELOG.md, docs/roadmap.md	Removes the methods from ApiClient, adds them to the API surface checks, lowers the client file-size limit, and records SCALE-SEAM (99).

Estimated code review effort: 2 (Simple) | ~10 minutes

Suggested reviewers: cursoragent

✨ Finishing Touches

📝 Generate docstrings

• Create stacked PR
• Commit on current branch

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Commit unit tests in branch claude/codebase-audit-roadmap-45656d

---

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

[ibuilder]

@coderabbitai review

---

Generated by Claude Code

[coderabbitai]

Tip

For best results, initiate chat on the files or code changes.

🧩 Analysis chain

🏁 Script executed:

#!/bin/bash
set -euo pipefail

echo '== Pull request metadata =='
gh pr view 432 --repo ibuilder/massing --json number,title,baseRefName,headRefName,changedFiles,additions,deletions,files

echo '== Changed-file diff summary =='
git diff --stat origin/main...HEAD
git diff --find-renames --unified=80 origin/main...HEAD -- \
  apps/web/src/api/client.ts \
  apps/web/src/api/authoring.ts \
  apps/web/src/api \
  CHANGELOG.md \
  docs/roadmap.md

echo '== API module outlines =='
ast-grep outline apps/web/src/api/client.ts --items all
ast-grep outline apps/web/src/api/authoring.ts --items all

echo '== Content API definitions and references =='
rg -n -C 4 --glob '!node_modules/**' \
  '\b(contentCatalog|placeContent|importContent)\b' \
  apps/web/src

echo '== API module import graph =='
rg -n -C 3 --glob '!node_modules/**' \
  'from\s+[\"\x27].*(client|authoring)[\"\x27]|import\s*\(' \
  apps/web/src

Length of output: 236715

---

🏁 Script executed:

#!/bin/bash
set -euo pipefail

echo '== Authoring mixin: content and family methods =='
sed -n '250,445p' apps/web/src/api/authoring.ts

echo '== API tests related to the authoring mixin =='
fd -a '^authoring.*\.test\.ts$|^surface\.test\.ts$|^clientCallers\.test\.ts$' apps/web/src/api \
  -x sh -c 'echo "---- $1"; sed -n "1,340p" "$1"' sh {}

echo '== Changed file-size guard =='
git diff --unified=30 origin/main...HEAD -- services/api/test_file_sizes.py
sed -n '1,260p' services/api/test_file_sizes.py

echo '== Diff integrity and current changed lines =='
git diff --check origin/main...HEAD
git diff --numstat origin/main...HEAD
git status --short

echo '== Pull request checks and review state =='
gh pr view 432 --repo ibuilder/massing --json statusCheckRollup,reviewDecision,isDraft,mergeStateStatus

Length of output: 183442

---

@ibuilder: Review complete. I found no actionable issues.

The three methods retain their signatures and request behavior. ApiClient still exposes them through withAuthoring(HttpCore). The existing content-library UI still calls all three methods. The surface test now protects the moved methods, and the client.ts size limit matches the new 648-line count.

git diff --check reported no whitespace errors. The required CI checks were still in progress when I inspected the pull request.

You are interacting with an AI system.

[strix-security]

Strix is installed on this repository, but we couldn't run this PR security review because this workspace's trial has ended. Add a card to resume code reviews here.

So far, Strix has reviewed 28 pull requests, surfaced 3 security issues (1 critical/high) and blocked 2 risky merges across this workspace.

[qodo-code-review]

ⓘ Qodo reviews are paused because your trial has ended. Ask your workspace admin to add credits to resume reviews. Manage billing