Add command to remove entry from history.

[Hofer-Julian]

Add command to remove entry from history.

[fdncred]

This one was written in nu_script by @giggio #11470 (comment)
https://github.com/giggio/nuscripts/blob/main/scripts/history-command.nu

# Deletes a history entry.
def 'history delete' [
  id: int = 0 # the id of the history entry to delete
  --last # delete the last entry, ignore the id, if passed
  ] {
  if $last {
    open $nu.history-path | query db $"delete from history where id = \(select id from \(select id from history order by id desc LIMIT 2) order by id asc LIMIT 1)"
  } else {
    if $id == 0 {
      echo "You must pass an id or use --last"
      exit 1
    }
    open $nu.history-path | query db $"delete from history where id = ($id)"
  }
  null
}

[Hofer-Julian]

Are you suggesting adding it to std, or did you just mention it for inspiration?

[fdncred]

Are you suggesting adding it to std, or did you just mention it for inspiration?

Either? I've never really used it, but it looks sound. Is this how you'd want a history remove-item to work?

[Hofer-Julian]

Is this how you'd want a history remove-item to work?

I think so?
To be honest I don't fully understand what this does:
\(select id from \(select id from history order by id desc LIMIT 2) order by id asc LIMIT 1)

I've assumed simply grabbing the entry with the correct id would be enough

[fdncred]

ok.

ya, i'm not sure why the LIMIT 2 and then LIMIT 1. I try to stay away from nested queries. @gigggio i assume you're doing that for some specific reason that we aren't seeing. can you provide any insight?

[giggio]

Yes, there is a reason. That is for deleting the last command. But the command you just entered became the last, so what you actually need to do is delete the one before last. That is what that does. As the first query is limiting to 2 rows only, I didn't see a problem to nest. If there is a better way to get the row before the last one I'd like to know, I can update my version, too.

And now that I'm saying that I realize that maybe you'd want to delete the last from your current session, and I'm not doing that filtering. I think I'll update my script. :)

[fdncred]

Please share your script if you decide to update it.

[giggio]

I just updated it:

https://github.com/giggio/nuscripts/blob/067c0479e876fd93825e793715bd5ddffcbefe4f/scripts/history-command.nu#L19-L34

[fdncred]

nice! Thanks @giggio - just adding it here for fun. I understand why you do the query for --last now. Good work!

# Deletes a history entry.
def 'history delete' [
  id: int = 0 # the id of the history entry to delete
  --last # delete the last entry, ignore the id, if passed
  ] {
  if $last {
    open $nu.history-path | query db $"delete from history where id = \(select id from \(select id from history where session_id = (history session) order by id desc LIMIT 2) order by id asc LIMIT 1)"
  } else {
    if $id == 0 {
      echo "You must pass an id or use --last"
      exit 1
    }
    open $nu.history-path | query db $"delete from history where id = ($id)"
  }
  null
}