Add a transcript showing that #5178 is fixed

[sellout]

@source{some ability member} now shows the source of the ability.

Fixes #5178.

[sellout]

Here it is rendered in the UI:

[aryairani]

Sorry for the delay, I'm having some trouble testing this locally. I wanted to just add a > display foo to the transcript, but I'm getting this:

scratch/main> display foo

      -- The name #rfi1v9429f is assigned to the reference
      ShortHash {prefix =
      "rfi1v9429f9qluv533l2iba77aadttilrpmnhljfapfnfa6sru2nr8ibpqvib9nc4s4nb9s1as45upsfqfqe6ivqi2p82b2vd866it8",
      cycle = Nothing, cid = Nothing}, which is missing from the
      codebase.
      Tip: You might need to repair the codebase manually.

Is that expected?

Maybe user error though.

[aryairani]

Confirmed that it displays okay in ui

but not with display

[aryairani]

So, I think it might not be fixed, but the transcript is still good for when we do fix it; we should add the display line though

eg

```unison
foo = {{
@source{Stream.emit}
}}
```

```ucm
scratch/main> add
scratch/main> view foo
scratch/main> display foo
```
```

[sellout]

Oh, thanks for catching this! From the original issue, I thought the error was happening during typechecking (and maybe it had been?), so I figured even ui was just a sanity check.

display certainly makes more sense for the transcript, but I’m surprised it and ui differ deeply enough for only one to expose this failure.

[aryairani]

I’m surprised it and ui differ deeply enough for only one to expose this failure.

Me too! cc @ChrisPenner if you happen to know why

[sellout]

Do we have any way to track “known failures” in transcripts or otherwise?

This is different than “unexpected success”, as “unexpected success” are tests that are supposed to fail, whereas “known failures” are tests that should succeed but involve known bugs (like the one here).

The salient difference being that when I see “unexpected success” my reaction is “What did I break?”, but with “fixed a known failure” my reaction is “Sweet! We already have a test case for this!”

[sellout]

scratch/main> display foo

      -- The name #rfi1v9429f is assigned to the reference
      ShortHash {prefix =
      "rfi1v9429f9qluv533l2iba77aadttilrpmnhljfapfnfa6sru2nr8ibpqvib9nc4s4nb9s1as45upsfqfqe6ivqi2p82b2vd866it8",
      cycle = Nothing, cid = Nothing}, which is missing from the
      codebase.
      Tip: You might need to repair the codebase manually.

An interesting thing with this is that display isn’t failing. UCM returns a successful result, but the content displayed is an error message.

[aryairani]

Do we have any way to track “known failures” in transcripts or otherwise?

This is different than “unexpected success”, as “unexpected success” are tests that are supposed to fail, whereas “known failures” are tests that should succeed but involve known bugs (like the one here).

We could add another directory for it and another mode to the transcripts executable.

The salient difference being that when I see “unexpected success” my reaction is “What did I break?”, but with “fixed a known failure” my reaction is “Sweet! We already have a test case for this!”

Makes sense!

An interesting thing with this is that display isn’t failing. UCM returns a successful result, but the content displayed is an error message.

Agreed, it does seem like this should be an error. I'm guessing that the error is being detected in OutputMessages which is probably an oversight.

[hojberg]

Converted to draft based on the above conversation that #5178 isn't fixed yet.

[ChrisPenner]

cc @ChrisPenner if you happen to know why

I'm afraid I don't have any insights at a glance here 🤔