@jdm @mrobinson would like to take on this task

@rohansx Ok! Take a look at the model PR in #31569 and ask any questions about the task here!

@servo-highfive assign me

Hey @MunishMummadi! Thanks for your interest in working on this issue. It's now assigned to you!

Hey @jdm . I would love to work on this issue. Can you mentor me for this issue. I understood what to do. But it seems little overwhelming.

Yes, I'm happy to answer questions about it

@jdm . I am sorry to say this. I tried my best. But I failed. Thank you for the heads up. You can unassign me.

@jdm Went through the issue and need a few clarifications:
The PR states
output of CGClassConstructHook (Present on Line 6111)
Does this class code needs to be changed for better readability ?

Because the highlighted code section in (difficult to read) shows code line numbers from 6175-6212

Thanks for checking!

@servo-highfive @jdm Just one more additional help needed here. I have performed some code changes for this PR.
However can you guide me as to how do I test this function while running this application in local setup? (For debugging purpose)

If the page loads like it did before, I would expect that's good enough for a PR. We can run the full automated test suite to verify the changes.

Hello @jdm attempted this issue and created PR #31949 but would like a litte guidance .
I'm struggling with interop (running rust code within python file codegenrust.py
My rust code is in a separate file called construct_hook.rs
Added the necessary import of crate in import.rs
Facing the below interop errors:
error[E0424]: expected value, found module 'self' --> /home/rsx/Desktop/outreachy/servo/target/debug/build/script-30067a5bb009356d/out/Bindings/FileReaderSyncBinding.rs:515:93 | 512 | unsafe extern fn _constructor(cx: *mut JSContext, argc: u32, vp: *mut JSVal) -> bool { | ------------ this function can't have aselfparameter ... 515 | let global = get_global_scope(cx, vp, argc);handle_constructor(cx, &argc, &global, &self.descriptor, &self.constructor); | ^^^^selfvalue is a keyword only available in methods with aself parameter

self.constructor is only available in python, rust cannot use python's variables.

Hello Thank you for the guidance made a bit more changes can you help me with this one last query
I noticed I can't find use crate import statements mentioned in PR https://github.com/servo/servo/pull/31569/files
I need to add this similar line to make my code work. Has the location been changed to import.rs

use crate::construct_hook::construct_if_block;

There were some changes to how importing is done. All you need to to is add proper use statements to import.rs. #31569 was made before this was introduced in #31711.

Hello @sagudev a bit more help please, struggling to make build here . In my rust code (which I have separated in construct_hook.rs facing the below error for PrototypeList::ID::"%s"
Do I have to explicitly define the type?
error: expected identifier, found "%s" --> components/script/dom/bindings/construct_hook.rs:18:28 | 18 | PrototypeList: id: "%s", // Adjust as needed | ^^^^ expected identifier Building [=======================> ] 762/768: script

I am not sure if I understand error, it better to use ``` blocks when having code of multiple lines. From what I see in PR you need to be more careful what is python and what is rust. Codegen (python code) generates rust code (this is usually done with strings manipulation).