Public: anyone can read this; the crew can change it.
The Absorb findings are written up for Ben in one short note, ready for the owner to send #9607
Check · Review · Land · Live done — Landed and live in production.
mason/forge:mason/9580-the-absorb-findings-are-written-up-for-b → main 2 commits fork 1 thread · 1 unresolved
1 file
changed +12
−0
docs/features/absorb.md +12
| @@ -71,3 +71,15 @@ See `docs/features/absorb-review.md` for the findings of #9441. | ||
| 71 | 71 | ## Not done yet |
| 72 | 72 | |
| 73 | 73 | The forge does not yet fire a feature's schedule, show its feed items, serve its MCP tools inside the forge's `/mcp`, keep `secret` settings or show its health on `/system`. |
| 74 | ||
| 75 | ## Findings | |
| 76 | ||
| 77 | Absorb was built by Ben (benjivers). The review of #9441 found: | |
| 78 | ||
| 79 | - **Security:** a push to the production branch can relabel an entry in every reader's menu (#9541). | |
| 80 | - **Without the crew:** it works with the crew off and no model credential, as `test/apps-features.test.ts` shows (#9441). | |
| 81 | - **Experience:** an admin has little to find Absorb by and no preview before it acts (#9544). | |
| 82 | - **Product:** a pack cannot say "absorb this app", so each needs a click-through (#9545). | |
| 83 | - **Speed:** the Features list reads git for every app on every request (#9543). | |
| 84 | ||
| 85 | This section is ready for the owner to share with Ben. |
Issue #9580 asks that every angle name its issue; this one names none. Add one (#9441 is the review that established it) instead of saying no issue was needed.