Skip to content

feat(lua-agent-builder): App Viewers in web apps, and /lua-audit flags their writes without an ownership check; 1.18.0 - #32

Merged
selcuk-lua merged 2 commits into
mainfrom
selcuk/lua-1898-app-viewer-guidance
Oct 29, 2026
Merged

selcuk-lua merged 2 commits into
mainfrom
selcuk/lua-1898-app-viewer-guidance

Conversation

@selcuk-lua

Copy link
Copy Markdown
Contributor

What

  • lib/knowledge/web-apps.md §3: auth.accessLevel ('full', 'view_only', 'use') and defineWebApp({ viewers: { write, chat, mode } }) (lua-cli 3.64.0+, lua-core-services #7071, gateway #7084). With write: true, App Viewers (people who sign in by phone or email to use only that app) reach every write route as 'use' with empty roles and scopes, so the builder is told, with an example route, that each write checks the record is the caller's (their userId, or their scope in the app's own roster) and answers 404 otherwise. chat and mode: 'app' are marked as not honoured yet.
  • agents/lua-auditor.md: viewers and accessLevel branches go under custom access control; new finding 14, App Viewer writes without ownership, always HIGH.
  • Release 1.18.0 (CHANGELOG, plugin/marketplace/MCP metadata, dist/server.js version strings).

Checks

npm run lint (incl. the version-consistency lint) and npm test: 72 suites, 2562 passed.

Linear: LUA-1898 (epic LUA-1892)

🤖 Generated with Claude Code

…s their writes without an ownership check; 1.18.0

web-apps.md: auth.accessLevel ('full', 'view_only', 'use') and
defineWebApp({ viewers: { write, chat, mode } }) (lua-cli 3.64.0+), with an
example write route that checks the record is the caller's and answers 404.
lua-auditor: lists viewers and accessLevel branches, and finding 14 (always
HIGH): viewers.write with a write route lacking an ownership check.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@selcuk-lua
selcuk-lua merged commit 5bba9b2 into main Oct 29, 2026
7 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.

1 participant