Add AlterIndex completion

[jachro]

This tiny PR follows the suggestion from the SkunkException I received after issuing an ALTER INDEX command.

I tried to follow the guidelines for running the tests locally but I hit one issue:

==> X tests.StartupTest.invalid port 0.01s scala.scalajs.js.JavaScriptException: AggregateError
    at <jscode>.internalConnectMultiple(node:net:1111)
    at <jscode>.afterConnectMultiple(node:net:1663)
    at delay @ <jscode>.{anonymous}()(/Users/chrobasi/sdsc/code/skunk/modules/tests/js/target/scala-2.13/tests-test-fastopt/main.js:93975)
    at as @ <jscode>.{anonymous}()(/Users/chrobasi/sdsc/code/skunk/modules/tests/js/target/scala-2.13/tests-test-fastopt/main.js:246615)
    at delay @ fs2.io.internal.facade.events.EventEmitter$ops$.registerOneTimeListener$extension(/Users/chrobasi/sdsc/code/skunk/modules/tests/js/target/scala-2.13/tests-test-fastopt/main.js:7947)
    at productL @ <jscode>.{anonymous}()(/Users/chrobasi/sdsc/code/skunk/modules/tests/js/target/scala-2.13/tests-test-fastopt/main.js:93980)
    at flatMap @ <jscode>.{anonymous}()(/Users/chrobasi/sdsc/code/skunk/modules/tests/js/target/scala-2.13/tests-test-fastopt/main.js:90305)
    at async @ <jscode>.{anonymous}()(/Users/chrobasi/sdsc/code/skunk/modules/tests/js/target/scala-2.13/tests-test-fastopt/main.js:93964)
    at allocatedCase @ <jscode>.{anonymous}()(/Users/chrobasi/sdsc/code/skunk/modules/tests/js/target/scala-2.13/tests-test-fastopt/main.js:46843)
    at refOf @ <jscode>.{anonymous}()(/Users/chrobasi/sdsc/code/skunk/modules/tests/js/target/scala-2.13/tests-test-fastopt/main.js:43986)
    at apply @ skunk.net.protocol.Exchange$.apply(/Users/chrobasi/sdsc/code/skunk/modules/tests/js/target/scala-2.13/tests-test-fastopt/main.js:41801)
    at allocatedCase @ <jscode>.{anonymous}()(/Users/chrobasi/sdsc/code/skunk/modules/tests/js/target/scala-2.13/tests-test-fastopt/main.js:46843)
    at refOf @ <jscode>.{anonymous}()(/Users/chrobasi/sdsc/code/skunk/modules/tests/js/target/scala-2.13/tests-test-fastopt/main.js:43986)
    at allocatedCase @ <jscode>.{anonymous}()(/Users/chrobasi/sdsc/code/skunk/modules/tests/js/target/scala-2.13/tests-test-fastopt/main.js:46843)
    at onError @ <jscode>.{anonymous}()(/Users/chrobasi/sdsc/code/skunk/modules/tests/js/target/scala-2.13/tests-test-fastopt/main.js:42665)
    at make @ skunk.util.Pool$.ofF(/Users/chrobasi/sdsc/code/skunk/modules/tests/js/target/scala-2.13/tests-test-fastopt/main.js:42801)
    at make @ skunk.util.Pool$.ofF(/Users/chrobasi/sdsc/code/skunk/modules/tests/js/target/scala-2.13/tests-test-fastopt/main.js:42801)

Unfortunately, I'm not sure how to fix it but I've got a feeling it's probably not related to my change.

[armanbilge]

I tried to follow the guidelines for running the tests locally but I hit one issue:

What version of Node.js do you have installed? i.e. node --version.

[jachro]

I tried to follow the guidelines for running the tests locally but I hit one issue:

What version of Node.js do you have installed? i.e. node --version.

I've got v20.5.0

[armanbilge]

Thanks, I opened a PR to fix it.

• Handle AggregateErrors on Node.js fs2#3269

[codecov-commenter]

Codecov Report

All modified and coverable lines are covered by tests ✅

Comparison is base (81c936c) 84.32% compared to head (5eec2e9) 84.33%.

Additional details and impacted files

@@              Coverage Diff              @@
##           series/0.6.x     #935   +/-   ##
=============================================
  Coverage         84.32%   84.33%           
=============================================
  Files               126      126           
  Lines              1735     1736    +1     
  Branches            170      161    -9     
=============================================
+ Hits               1463     1464    +1     
  Misses              272      272           

Files	Coverage Δ
...s/core/shared/src/main/scala/data/Completion.scala	100.00% <ø> (ø)
...d/src/main/scala/net/message/CommandComplete.scala	92.30% <100.00%> (+0.12%)	⬆️

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

[armanbilge]

This looks like a good candidate to re-target for series/0.6.x branch.

[jachro]

This looks like a good candidate to re-target for series/0.6.x branch.

I know it's a tiny change but it's still a pleasure to hear that.