Mercurial > public > mercurial-scm > hg-stable
diff hgext/show.py @ 33199:c5a07a3abe7d
show: implement "stack" view
People often want to know what they are working on *now*. As part of
this, they also commonly want to know how that work is related to other
changesets in the repo so they can perform common actions like rebase,
histedit, and merge.
`hg show work` made headway into this space. However, it is geared
towards a complete repo view as opposed to just the current line of
work. If you have a lot of in-flight work or the repo has many heads,
the output can be overwhelming. The closest thing Mercurial has to
"show me the current thing I'm working on" that doesn't require custom
revsets is `hg qseries`. And this requires MQ, which completely changes
workflows and repository behavior and has horrible performance on large
repos. But as sub-optimal as MQ is, it does some things right, such as
expose a model of the repo that is easy for people to reason about.
This simplicity is why I think a lot of people prefer to use MQ, despite
its shortcomings.
One common development workflow is to author a series of linear
changesets, using bookmarks, branches, anonymous heads, or even topics
(3rd party extension). I'll call this a "stack." You periodically
rewrite history in place (using `hg histedit`) and reparent the stack
against newer changesets (using `hg rebase`). This workflow can be
difficult because there is no obvious way to quickly see the current
"stack" nor its relation to other changesets. Figuring out arguments to
`hg rebase` can be difficult and may require highlighting and pasting
multiple changeset nodes to construct a command.
The goal of this commit is to make stack based workflows simpler
by exposing a view of the current stack and its relationship to
other releant changesets, notably the parent of the base changeset
in the stack and newer heads that the stack could be rebased or merged
into.
Introduced is the `hg show stack` view. Essentially, it finds all
mutable changesets from the working directory revision in both
directions, stopping at a merge or branch point. This limits the
revisions to a DAG linear range.
The stack is rendered as a concise list of changesets. Alongside the
stack is a visualization of the DAG, similar to `hg log -G`.
Newer public heads from the branch point of the stack are rendered
above the stack. The presence of these heads helps people understand
the DAG model and the relationship between the stack and changes made
since the branch point of that stack. If the "rebase" command is
available, a `hg rebase` command is printed for each head so a user
can perform a simple copy and paste to perform a rebase.
This view is alpha quality. There are tons of TODOs documented
inline. But I think it is good enough for a first iteration.
author | Gregory Szorc <gregory.szorc@gmail.com> |
---|---|
date | Sat, 01 Jul 2017 22:38:42 -0700 |
parents | 99ce2f586cd4 |
children | 9e7efe421395 |
line wrap: on
line diff
--- a/hgext/show.py Wed Jun 28 21:30:46 2017 -0400 +++ b/hgext/show.py Sat Jul 01 22:38:42 2017 -0700 @@ -32,9 +32,11 @@ from mercurial import ( cmdutil, commands, + destutil, error, formatter, graphmod, + phases, pycompat, registrar, revset, @@ -171,6 +173,166 @@ fm.data(active=bm == active, longestbookmarklen=longestname) +@showview('stack', csettopic='stack') +def showstack(ui, repo, displayer): + """current line of work""" + wdirctx = repo['.'] + if wdirctx.rev() == nullrev: + raise error.Abort(_('stack view only available when there is a ' + 'working directory')) + + if wdirctx.phase() == phases.public: + ui.write(_('(empty stack; working directory is a published ' + 'changeset)\n')) + return + + # TODO extract "find stack" into a function to facilitate + # customization and reuse. + + baserev = destutil.stackbase(ui, repo) + basectx = None + + if baserev is None: + baserev = wdirctx.rev() + stackrevs = {wdirctx.rev()} + else: + stackrevs = set(repo.revs('%d::.', baserev)) + + ctx = repo[baserev] + if ctx.p1().rev() != nullrev: + basectx = ctx.p1() + + # And relevant descendants. + branchpointattip = False + cl = repo.changelog + + for rev in cl.descendants([wdirctx.rev()]): + ctx = repo[rev] + + # Will only happen if . is public. + if ctx.phase() == phases.public: + break + + stackrevs.add(ctx.rev()) + + if len(ctx.children()) > 1: + branchpointattip = True + break + + stackrevs = list(reversed(sorted(stackrevs))) + + # Find likely target heads for the current stack. These are likely + # merge or rebase targets. + if basectx: + # TODO make this customizable? + newheads = set(repo.revs('heads(%d::) - %ld - not public()', + basectx.rev(), stackrevs)) + else: + newheads = set() + + try: + cmdutil.findcmd('rebase', commands.table) + haverebase = True + except error.UnknownCommand: + haverebase = False + + # TODO use templating. + # TODO consider using graphmod. But it may not be necessary given + # our simplicity and the customizations required. + # TODO use proper graph symbols from graphmod + + shortesttmpl = formatter.maketemplater(ui, '{shortest(node, 5)}') + def shortest(ctx): + return shortesttmpl.render({'ctx': ctx, 'node': ctx.hex()}) + + # We write out new heads to aid in DAG awareness and to help with decision + # making on how the stack should be reconciled with commits made since the + # branch point. + if newheads: + # Calculate distance from base so we can render the count and so we can + # sort display order by commit distance. + revdistance = {} + for head in newheads: + # There is some redundancy in DAG traversal here and therefore + # room to optimize. + ancestors = cl.ancestors([head], stoprev=basectx.rev()) + revdistance[head] = len(list(ancestors)) + + sourcectx = repo[stackrevs[-1]] + + sortedheads = sorted(newheads, key=lambda x: revdistance[x], + reverse=True) + + for i, rev in enumerate(sortedheads): + ctx = repo[rev] + + if i: + ui.write(': ') + else: + ui.write(' ') + + ui.write(('o ')) + displayer.show(ctx) + displayer.flush(ctx) + ui.write('\n') + + if i: + ui.write(':/') + else: + ui.write(' /') + + ui.write(' (') + ui.write(_('%d commits ahead') % revdistance[rev], + label='stack.commitdistance') + + if haverebase: + # TODO may be able to omit --source in some scenarios + ui.write('; ') + ui.write(('hg rebase --source %s --dest %s' % ( + shortest(sourcectx), shortest(ctx))), + label='stack.rebasehint') + + ui.write(')\n') + + ui.write(':\n: ') + ui.write(_('(stack head)\n'), label='stack.label') + + if branchpointattip: + ui.write(' \\ / ') + ui.write(_('(multiple children)\n'), label='stack.label') + ui.write(' |\n') + + for rev in stackrevs: + ctx = repo[rev] + symbol = '@' if rev == wdirctx.rev() else 'o' + + if newheads: + ui.write(': ') + else: + ui.write(' ') + + ui.write(symbol, ' ') + displayer.show(ctx) + displayer.flush(ctx) + ui.write('\n') + + # TODO display histedit hint? + + if basectx: + # Vertically and horizontally separate stack base from parent + # to reinforce stack boundary. + if newheads: + ui.write(':/ ') + else: + ui.write(' / ') + + ui.write(_('(stack base)'), '\n', label='stack.label') + ui.write(('o ')) + + displayer.show(basectx) + displayer.flush(basectx) + ui.write('\n') + @revsetpredicate('_underway([commitage[, headage]])') def underwayrevset(repo, subset, x): args = revset.getargsdict(x, 'underway', 'commitage headage')