Skip to content

Share posted deployment links with their scope - #658

Closed
16francej wants to merge 1 commit into
mainfrom
fix-shared-deploy-links
Closed

Share posted deployment links with their scope#658
16francej wants to merge 1 commit into
mainfrom
fix-shared-deploy-links

Conversation

@16francej

@16francej 16francej commented Aug 21, 2026

Copy link
Copy Markdown
Contributor

When the owner of a published app posts its URL into a shared channel or group conversation, grant that scope read access so every member can reach it. Lookalike URLs and non-owner posts are ignored.

  • verification: node --test test/deployment-link-share.test.ts (new regressions) plus existing deployment authz suites pass; typecheck and lint clean

View with [code]smith Autofix with [code]smith
Need help on this PR? Tag @codesmith-bot with what you need. Autofix is disabled.

Co-Authored-By: QM <qm@ycombinator.com>
@16francej
16francej force-pushed the fix-shared-deploy-links branch from e473de2 to 4afe104 Compare August 21, 2026 23:40
@time-attack

Copy link
Copy Markdown
Collaborator

Thanks for this, @16francej — and sorry for the slow response.

We're closing this PR. Per CONTRIBUTING.md, since coding agents write most of the underlying code now, we'd rather receive feature ideas as a short human-written note in adrs/ — informal is great, just the idea in your own words — and bugs as a plain issue. If we're aligned we're happy to burn our own tokens on the implementation, and we'll credit you as co-author.

We don't want your idea to get lost, so we've captured it in #751 to triage on our side. Please feel free to open an adrs/ note if you feel strongly about the approach. Appreciate you engaging with the repo!

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