feat!: clean up internal symbols usage

[florian-lefebvre]

Changes

• The astro.locals symbol is only used in dev by official adapters, so it can be removed from here
• the astro.clientAddress symbol is meant to be used internally only, now that clientAddress can be passed to app.render()

Testing

Should still pass

Docs

Shouldn't be needed as it's an undocumented API

[changeset-bot]

⚠️ No Changeset found

Latest commit: 74ed9d4

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.

Click here to learn what changesets are, and how to add one.

Click here if you're a maintainer who wants to add a changeset to this PR

[florian-lefebvre]

@sarah11918 let me know if you think this needs docs but I don't think it does!

[florian-lefebvre]

Updated to match cloudflare

[sarah11918]

I think this all seems pretty internal, if it's only official adapters (so, integration authors don't need to know/do anything different?) and it's otherwise internal code (so, no Astro project code needs to change.)

So, I'll still want to review the changeset in the end, but likely no docs seems like the right call!

[florian-lefebvre]

Since it should only be a refactor, I'm thinking about not including a changeset at all