feat: comments, notifications, role-binding authorization - #23
Merged
Merged
Conversation
Snapshot of the private repository at bade87e (version 0.2.0). Carries: - fix: improve header and sharing settings layers (#187) - fix: complete sharing guidance and viewer polish (#186) - fix: preserve native preview anchor activation (#185) - feat: refine sharing and account navigation (#182) - feat: prepare public pages for promotion (#184) - fix: keep preview fragment links in the loaded document (#183) - fix: preserve users across OIDC subject changes (#181) - feat: add permission-aware comment mentions (#179) - feat: progressively load visible artifact previews (#180) - feat: add discussion follows and collaboration notifications (#177) - Clarify comment destinations and simplify share links (#178) - fix: eliminate directory permission and navigation waterfalls (#176) - style: keep the home update link beside its hint (#175) - Improve comment polling, selection placement and update guidance (#174) - feat: add version uploads and contextual viewer guidance (#173) - fix: harden comment images and agent result attribution (#172) - fix: address artifact navigation and agent library feedback (#171) - feat: polish comment discovery and image feedback (#170) - feat: search discussions and link comment results to versions (#169) - feat: add rich comment text and private image attachments (#168) - refactor: simplify contextual comment composition (#167) - feat: improve comment creation and reading (P0) (#166) - fix: preserve upload receipts during concurrent PUTs (#165) - feat: add Agent comment context and same-site revision workflow (#163) - fix: keep site action menus within the viewport (#162) - refactor: remove retired authorization storage and interfaces (#160) - docs: restore original README banner width in all languages (#161) - docs: roadmap after 0.2.0 (#159) - feat: migrate authorization to database role bindings (#157) Source: artifact-hub bade87eed0d826fff2c8669009f5144211b3cbb4 Signed-off-by: xiongxz <xiuzhang.xiong@lexmount.com>
Merged
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Full snapshot of the private repository at
bade87e, bringing the public tree level with it. 26 private pull requests since the 0.2.0 snapshot; #159, #161 and #165 were already cherry-picked here and are unchanged by this sync.What it carries
permissions,roles,role_permissionsandrole_bindingsreplace hard-coded runtime roles and legacy records; retired storage and the redundant collaborator/edit-policy interface are removed. Migrations 0007–0009 plus a bootstrap step. Deploy with a coordinated restart — old and new authorization writers must not run together (seedocs/RBAC.md).robots.txtand a marketing-onlysitemap.xml, localized quota guidance. Artifact pages keep their policy-aware cards and stay out of search results.Four new migrations (0010 comment attachments, 0011 comment results, 0012 notifications, 0013 comment mentions) and two new settings documented in
.env.exampleandSELFHOST.md. Nothing hard-codes a hosted domain; metadata, robots and sitemap all derive from the deployment's own public URL.Verified locally on this branch
npm ci(root andcli/),tsc --noEmit,eslint .,vitest run(165 files, 1874 tests passed, 23 files skipped),clitests, andnext build— all green. The export excludesdocs/internal, deployment kit files and the sync script, as before.Suggested follow-up: tag
v0.3.0after merge.🤖 Generated with Claude Code