fix: Fix GitHub release description (#26247)

[Gudahtt]

This is a cherry-pick of #26247 for v12.0.1. Original description:

Description

For years our release scripts have been creating GitHub releases with empty descriptions. GitHub uses the commit message as the description as a default, which is always a merge commit in our case, whcih isn't very decsriptive. The release scripts are supposed to be setting the changelog entries for the current release as the description.

This broke when we last changed our CHANGELOG.md format. Specifically it was the release header change that broke this. We started surrounding the version number in square brackets, which the awk script did not expect.

The script that creates the GitHub release has been updated to pass in a version surrounded by square brackets to the awk script, so it is again able to find the relevant changelog entries.

Related issues

Fixes: #18468

Manual testing steps

Start by checking out master. Then look at the release-create-gh-release.shscript, and run the exact same awk command that is shown there. Replace${tag##v}with the release version minus the v (e.g.12.0.0`).

Before this PR it was awk -v version="12.0.0" -f .circleci/scripts/show-changelog.awk CHANGELOG.md. This produces no output.

After this PR, its awk -v version="[12.0.0]" -f .circleci/scripts/show-changelog.awk CHANGELOG.md. This produces the correct output.

Screenshots/Recordings

N/A

Pre-merge author checklist

• I've followed MetaMask Contributor Docs and MetaMask Extension Coding
  Standards.
• I've completed the PR template to the best of my ability
• I’ve included tests if applicable
• I’ve documented my code using JSDoc format if applicable
• I’ve applied the right labels on the PR (see labeling guidelines). Not required for external contributors.

Pre-merge reviewer checklist

• I've manually tested the PR (e.g. pull and build branch, run the app, test code being changed).
• I confirm that this PR addresses all acceptance criteria described in the ticket it closes and includes the necessary testing evidence such as recordings and or screenshots.

[metamaskbot]

Builds ready [d666b41]

• builds: chrome, firefox
• builds (beta): chrome
• builds (flask): chrome, firefox
• builds (MMI): chrome, firefox
• builds (test): chrome, firefox
• builds (test-flask): chrome, firefox
• build viz: Build System
• mv3: Background Module Init Stats
• mv3: UI Init Stats
• mv3: Module Load Stats
• mv3: Bundle Size Stats
• mv2: E2e Actions Stats
• code coverage: Report
• storybook: Storybook
• typescript migration: Dashboard
• all artifacts
• bundle viz:

  • background: 0, 1, 2, 3, 4, 5, 6
  • common: 0, 1, 2, 3, 4, 5, 6, 7, 8, 9
  • content-script: 0
  • offscreen: 0
  • ui: 0, 1, 10, 2, 3, 4, 5, 6, 7, 8, 9

Page Load Metrics (41 ± 2 ms)

Platform	Page	Metric	Min (ms)	Max (ms)	Average (ms)	StandardDeviation (ms)	MarginOfError (ms)
Chrome	Home	firstPaint	58	133	75	15	7
		domContentLoaded	8	15	9	1	1
		load	39	50	41	3	2
		domInteractive	8	15	9	1	1

[codecov]

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 65.61%. Comparing base (c880e07) to head (d666b41).

Additional details and impacted files

@@               Coverage Diff                @@
##           Version-v12.0.1   #26279   +/-   ##
================================================
  Coverage            65.61%   65.61%           
================================================
  Files                 1368     1368           
  Lines                54600    54600           
  Branches             14183    14183           
================================================
  Hits                 35825    35825           
  Misses               18775    18775           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

[desi]

Tested - LGTM

[DDDDDanica]

Tested in Version-v12.0.1 branch. ✅