fix: optimizer is now better at recognizing constProp

[JerryWu1234]

What is it?

fixes: #7216

Optimize the Optimize, has been better at recognizing constProp

Description

Checklist

• My code follows the developer guidelines of this project
• I performed a self-review of my own code
• I added a changeset with pnpm change
• I made corresponding changes to the Qwik docs
• I added new tests to cover the fix / functionality

[changeset-bot]

🦋 Changeset detected

Latest commit: 82d24fa

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 5 packages

Name	Type
@qwik.dev/core	Patch
eslint-plugin-qwik	Patch
@qwik.dev/react	Patch
@qwik.dev/router	Patch
create-qwik	Patch

Not sure what this means? Click here to learn what changesets are.

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

[pkg-pr-new]

Open in Stackblitz

npm i https://pkg.pr.new/QwikDev/qwik/@qwik.dev/core@7316

npm i https://pkg.pr.new/QwikDev/qwik/@qwik.dev/router@7316

npm i https://pkg.pr.new/QwikDev/qwik/eslint-plugin-qwik@7316

npm i https://pkg.pr.new/QwikDev/qwik/create-qwik@7316

commit: 82d24fa

[github-actions]

built with Refined Cloudflare Pages Action

⚡ Cloudflare Pages Deployment

Name	Status	Preview	Last Commit
qwik-docs	✅ Ready (View Log)	Visit Preview	82d24fa

[wmertens]

Good start, but it looks like _create_synthetic_qsegment is returning is_const=false because it's a simple identifier, and it doesn't know if the identifier is const.

So we need to pass the const identifiers to that function.

It's important to get this right because putting a handler in a variable before passing it as a handler is very common.

[wmertens]