fix: fix undefined error and remove unused files

[AayushSaini101]

Resolves: #1324

[AayushSaini101]

@derberg There was some issue in the voting, I have fixed and tested, #1323 we need to remove unwanted files

[derberg]

we need to make sure this new workflow can also be triggered manually:

  workflow_dispatch:
    inputs:
      bot_comment_url:
        description: |
          Provide URL pointing to gitvote bot comment that contains closing voting update. It looks like `https://github.com/asyncapi/community/issues/1313#issuecomment-2247595858`. We use this to update the voting summary in cases when we see errors in the voting status, when for example TSC member voted, but did a mistake and voted by adding emoji to main description or other bot comment instead of the correct way: which is adding an emoji to a comment from bot that opens the vote.
        required: true

in your script you need to check if the workflow was run manualy, parse the URL and extract data and use GitHub API GET /repos/{owner}/{repo}/issues/comments/{comment_id} to fetch the comment data and pass it to the script for later processing

[derberg]

most important changes that need to be done are here

#1325 (review)

and regarding the package and package-lock that were pushed to master by mistake, you need to remove them with this PR

[derberg]

Suggested change

	run: npm install @octokit/rest js-yaml@4.1.0 --no-save
	run: npm install js-yaml@4.1.0 --no-save

it is not needed, you don't even use it

GH rest is part of github context

[derberg]

github not githuh

and once you do it, later instead of const { Octokit } = await import("@octokit/rest"); you can do github.request

github-script action comes with authenticated rest client -> https://github.com/actions/github-script?tab=readme-ov-file. I think you used it in some other pr already

[AayushSaini101]

Removed Octokit used @derberg

[derberg]

this is very risky approach and error prone long term, you basically do not know what happens without reading the code of fetch. I mean sharing variables like this, end mutating them across different functions

instead fetchCommentInformation should return an object with all needed info, like orgName for example, and here you should assign returned values to variables

const voteCommentContext = await fetchCommentInformation();
orgName = voteCommentContext.orgName;

[AayushSaini101]

Done @derberg

[AayushSaini101]

@derberg Need little bit testing more i will revert once done

[aeworxet]

@asyncapi/bounty_team

[AayushSaini101]

@derberg you can review now done with testing https://github.com/AayushSaini101/Vote/blob/main/TSC_VOTING_OVERVIEW.md

[derberg]

/rtm