Shebang bug fix in launcher script

[jgao54]

Cannot have newline before shebang.

Type of change

• Bug fix

Test plan

• Manual testing: before this change, an image that uses this js_image_layer was still getting exec format error when trying to execute the js binary, which is expected since shebang cannot have newline in front of it. Error is fixed with this change.
• Updated unit & e2e tests.

[CLAassistant]

All committers have signed the CLA.

[jbedard]

@thesayyn do you know any way to test this?

[thesayyn]

can be added here

rules_js/e2e/js_image_oci/src/test.yaml

Lines 4 to 17 in bde7389

	- name: 'smoke'
	command: 'bash'
	args: ['/app/src/main']
	expectedOutput:
	[
	'OS',
	'ARCH/CPU',
	' WORKSPACE js_image_oci',
	' TARGET //src:main',
	' SOURCE CHECK true',
	' DIRECTORY CHECK true',
	' PROTO CHECK true',
	' SOURCE DIRECTORY CHECK true',
	]

instead of invoking it via bash, script can be invoked directly to see if it works directly.

something like

- name: 'smoke'
   entrypoint: /app/src/main

[thesayyn]

LGTM

[jgao54]

I'm seeing: line 9: field entrypoint not found in type v2.CommandTest errors, going to undo the e2e test change for now.

[jgao54]

@thesayyn will need another approval, thanks!

[thesayyn]

I'm seeing: line 9: field entrypoint not found in type v2.CommandTest errors, going to undo the e2e test change for now.

sorry it's supposed to be command.

[jgao54]

@thesayyn updated again.

[jgao54]

The e2e failures look unrelated to my change now.