releasenotes.py
602 lines
| 19.6 KiB
| text/x-python
|
PythonLexer
/ hgext / releasenotes.py
Gregory Szorc
|
r32778 | # Copyright 2017-present Gregory Szorc <gregory.szorc@gmail.com> | ||
# | ||||
# This software may be used and distributed according to the terms of the | ||||
# GNU General Public License version 2 or any later version. | ||||
"""generate release notes from commit messages (EXPERIMENTAL) | ||||
It is common to maintain files detailing changes in a project between | ||||
releases. Maintaining these files can be difficult and time consuming. | ||||
The :hg:`releasenotes` command provided by this extension makes the | ||||
process simpler by automating it. | ||||
""" | ||||
from __future__ import absolute_import | ||||
Rishabh Madan
|
r33883 | import difflib | ||
Gregory Szorc
|
r32778 | import errno | ||
import re | ||||
import sys | ||||
import textwrap | ||||
from mercurial.i18n import _ | ||||
from mercurial import ( | ||||
Rishabh Madan
|
r33572 | config, | ||
Gregory Szorc
|
r32778 | error, | ||
minirst, | ||||
registrar, | ||||
scmutil, | ||||
Rishabh Madan
|
r33572 | util, | ||
Gregory Szorc
|
r32778 | ) | ||
cmdtable = {} | ||||
command = registrar.command(cmdtable) | ||||
# Note for extension authors: ONLY specify testedwith = 'ships-with-hg-core' for | ||||
# extensions which SHIP WITH MERCURIAL. Non-mainline extensions should | ||||
# be specifying the version(s) of Mercurial they are tested with, or | ||||
# leave the attribute unspecified. | ||||
testedwith = 'ships-with-hg-core' | ||||
DEFAULT_SECTIONS = [ | ||||
('feature', _('New Features')), | ||||
('bc', _('Backwards Compatibility Changes')), | ||||
('fix', _('Bug Fixes')), | ||||
('perf', _('Performance Improvements')), | ||||
('api', _('API Changes')), | ||||
] | ||||
RE_DIRECTIVE = re.compile('^\.\. ([a-zA-Z0-9_]+)::\s*([^$]+)?$') | ||||
Rishabh Madan
|
r33661 | RE_ISSUE = r'\bissue ?[0-9]{4,6}(?![0-9])\b' | ||
Gregory Szorc
|
r32778 | |||
BULLET_SECTION = _('Other Changes') | ||||
class parsedreleasenotes(object): | ||||
def __init__(self): | ||||
self.sections = {} | ||||
def __contains__(self, section): | ||||
return section in self.sections | ||||
def __iter__(self): | ||||
return iter(sorted(self.sections)) | ||||
def addtitleditem(self, section, title, paragraphs): | ||||
"""Add a titled release note entry.""" | ||||
self.sections.setdefault(section, ([], [])) | ||||
self.sections[section][0].append((title, paragraphs)) | ||||
def addnontitleditem(self, section, paragraphs): | ||||
"""Adds a non-titled release note entry. | ||||
Will be rendered as a bullet point. | ||||
""" | ||||
self.sections.setdefault(section, ([], [])) | ||||
self.sections[section][1].append(paragraphs) | ||||
def titledforsection(self, section): | ||||
"""Returns titled entries in a section. | ||||
Returns a list of (title, paragraphs) tuples describing sub-sections. | ||||
""" | ||||
return self.sections.get(section, ([], []))[0] | ||||
def nontitledforsection(self, section): | ||||
"""Returns non-titled, bulleted paragraphs in a section.""" | ||||
return self.sections.get(section, ([], []))[1] | ||||
def hastitledinsection(self, section, title): | ||||
return any(t[0] == title for t in self.titledforsection(section)) | ||||
def merge(self, ui, other): | ||||
"""Merge another instance into this one. | ||||
This is used to combine multiple sources of release notes together. | ||||
""" | ||||
for section in other: | ||||
Rishabh Madan
|
r33661 | existingnotes = converttitled(self.titledforsection(section)) + \ | ||
convertnontitled(self.nontitledforsection(section)) | ||||
Gregory Szorc
|
r32778 | for title, paragraphs in other.titledforsection(section): | ||
if self.hastitledinsection(section, title): | ||||
# TODO prompt for resolution if different and running in | ||||
# interactive mode. | ||||
ui.write(_('%s already exists in %s section; ignoring\n') % | ||||
(title, section)) | ||||
continue | ||||
Rishabh Madan
|
r33661 | incoming_str = converttitled([(title, paragraphs)])[0] | ||
if section == 'fix': | ||||
issue = getissuenum(incoming_str) | ||||
if issue: | ||||
if findissue(ui, existingnotes, issue): | ||||
continue | ||||
if similar(ui, existingnotes, incoming_str): | ||||
continue | ||||
Gregory Szorc
|
r32778 | self.addtitleditem(section, title, paragraphs) | ||
for paragraphs in other.nontitledforsection(section): | ||||
if paragraphs in self.nontitledforsection(section): | ||||
continue | ||||
Rishabh Madan
|
r33661 | incoming_str = convertnontitled([paragraphs])[0] | ||
if section == 'fix': | ||||
issue = getissuenum(incoming_str) | ||||
if issue: | ||||
if findissue(ui, existingnotes, issue): | ||||
continue | ||||
if similar(ui, existingnotes, incoming_str): | ||||
continue | ||||
Gregory Szorc
|
r32778 | self.addnontitleditem(section, paragraphs) | ||
class releasenotessections(object): | ||||
Rishabh Madan
|
r33572 | def __init__(self, ui, repo=None): | ||
if repo: | ||||
sections = util.sortdict(DEFAULT_SECTIONS) | ||||
custom_sections = getcustomadmonitions(repo) | ||||
if custom_sections: | ||||
sections.update(custom_sections) | ||||
self._sections = list(sections.iteritems()) | ||||
else: | ||||
self._sections = list(DEFAULT_SECTIONS) | ||||
Gregory Szorc
|
r32778 | |||
def __iter__(self): | ||||
return iter(self._sections) | ||||
def names(self): | ||||
return [t[0] for t in self._sections] | ||||
def sectionfromtitle(self, title): | ||||
for name, value in self._sections: | ||||
if value == title: | ||||
return name | ||||
return None | ||||
Rishabh Madan
|
r33661 | def converttitled(titledparagraphs): | ||
""" | ||||
Convert titled paragraphs to strings | ||||
""" | ||||
string_list = [] | ||||
for title, paragraphs in titledparagraphs: | ||||
lines = [] | ||||
for para in paragraphs: | ||||
lines.extend(para) | ||||
string_list.append(' '.join(lines)) | ||||
return string_list | ||||
def convertnontitled(nontitledparagraphs): | ||||
""" | ||||
Convert non-titled bullets to strings | ||||
""" | ||||
string_list = [] | ||||
for paragraphs in nontitledparagraphs: | ||||
lines = [] | ||||
for para in paragraphs: | ||||
lines.extend(para) | ||||
string_list.append(' '.join(lines)) | ||||
return string_list | ||||
def getissuenum(incoming_str): | ||||
""" | ||||
Returns issue number from the incoming string if it exists | ||||
""" | ||||
issue = re.search(RE_ISSUE, incoming_str, re.IGNORECASE) | ||||
if issue: | ||||
issue = issue.group() | ||||
return issue | ||||
def findissue(ui, existing, issue): | ||||
""" | ||||
Returns true if issue number already exists in notes. | ||||
""" | ||||
if any(issue in s for s in existing): | ||||
ui.write(_('"%s" already exists in notes; ignoring\n') % issue) | ||||
return True | ||||
else: | ||||
return False | ||||
def similar(ui, existing, incoming_str): | ||||
""" | ||||
Returns true if similar note found in existing notes. | ||||
""" | ||||
if len(incoming_str.split()) > 10: | ||||
merge = similaritycheck(incoming_str, existing) | ||||
if not merge: | ||||
ui.write(_('"%s" already exists in notes file; ignoring\n') | ||||
% incoming_str) | ||||
return True | ||||
else: | ||||
return False | ||||
else: | ||||
return False | ||||
def similaritycheck(incoming_str, existingnotes): | ||||
""" | ||||
Returns true when note fragment can be merged to existing notes. | ||||
""" | ||||
import fuzzywuzzy.fuzz as fuzz | ||||
merge = True | ||||
for bullet in existingnotes: | ||||
score = fuzz.token_set_ratio(incoming_str, bullet) | ||||
if score > 75: | ||||
merge = False | ||||
break | ||||
return merge | ||||
Rishabh Madan
|
r33572 | def getcustomadmonitions(repo): | ||
ctx = repo['.'] | ||||
p = config.config() | ||||
def read(f, sections=None, remap=None): | ||||
if f in ctx: | ||||
data = ctx[f].data() | ||||
p.parse(f, data, sections, remap, read) | ||||
else: | ||||
raise error.Abort(_(".hgreleasenotes file \'%s\' not found") % | ||||
repo.pathto(f)) | ||||
if '.hgreleasenotes' in ctx: | ||||
read('.hgreleasenotes') | ||||
return p['sections'] | ||||
Rishabh Madan
|
r33883 | def checkadmonitions(ui, repo, directives, revs): | ||
""" | ||||
Checks the commit messages for admonitions and their validity. | ||||
.. abcd:: | ||||
First paragraph under this admonition | ||||
For this commit message, using `hg releasenotes -r . --check` | ||||
returns: Invalid admonition 'abcd' present in changeset 3ea92981e103 | ||||
As admonition 'abcd' is neither present in default nor custom admonitions | ||||
""" | ||||
for rev in revs: | ||||
ctx = repo[rev] | ||||
admonition = re.search(RE_DIRECTIVE, ctx.description()) | ||||
if admonition: | ||||
if admonition.group(1) in directives: | ||||
continue | ||||
else: | ||||
ui.write(_("Invalid admonition '%s' present in changeset %s" | ||||
"\n") % (admonition.group(1), ctx.hex()[:12])) | ||||
sim = lambda x: difflib.SequenceMatcher(None, | ||||
admonition.group(1), x).ratio() | ||||
similar = [s for s in directives if sim(s) > 0.6] | ||||
if len(similar) == 1: | ||||
ui.write(_("(did you mean %s?)\n") % similar[0]) | ||||
elif similar: | ||||
ss = ", ".join(sorted(similar)) | ||||
ui.write(_("(did you mean one of %s?)\n") % ss) | ||||
Rishabh Madan
|
r33942 | def _getadmonitionlist(ui, sections): | ||
for section in sections: | ||||
ui.write("%s: %s\n" % (section[0], section[1])) | ||||
Gregory Szorc
|
r32778 | def parsenotesfromrevisions(repo, directives, revs): | ||
notes = parsedreleasenotes() | ||||
for rev in revs: | ||||
ctx = repo[rev] | ||||
blocks, pruned = minirst.parse(ctx.description(), | ||||
admonitions=directives) | ||||
for i, block in enumerate(blocks): | ||||
if block['type'] != 'admonition': | ||||
continue | ||||
directive = block['admonitiontitle'] | ||||
title = block['lines'][0].strip() if block['lines'] else None | ||||
if i + 1 == len(blocks): | ||||
raise error.Abort(_('release notes directive %s lacks content') | ||||
% directive) | ||||
# Now search ahead and find all paragraphs attached to this | ||||
# admonition. | ||||
paragraphs = [] | ||||
for j in range(i + 1, len(blocks)): | ||||
pblock = blocks[j] | ||||
# Margin blocks may appear between paragraphs. Ignore them. | ||||
if pblock['type'] == 'margin': | ||||
continue | ||||
if pblock['type'] != 'paragraph': | ||||
raise error.Abort(_('unexpected block in release notes ' | ||||
'directive %s') % directive) | ||||
if pblock['indent'] > 0: | ||||
paragraphs.append(pblock['lines']) | ||||
else: | ||||
break | ||||
# TODO consider using title as paragraph for more concise notes. | ||||
if not paragraphs: | ||||
raise error.Abort(_('could not find content for release note ' | ||||
'%s') % directive) | ||||
if title: | ||||
notes.addtitleditem(directive, title, paragraphs) | ||||
else: | ||||
notes.addnontitleditem(directive, paragraphs) | ||||
return notes | ||||
def parsereleasenotesfile(sections, text): | ||||
"""Parse text content containing generated release notes.""" | ||||
notes = parsedreleasenotes() | ||||
blocks = minirst.parse(text)[0] | ||||
Rishabh Madan
|
r33012 | def gatherparagraphsbullets(offset, title=False): | ||
notefragment = [] | ||||
Gregory Szorc
|
r32778 | |||
for i in range(offset + 1, len(blocks)): | ||||
block = blocks[i] | ||||
if block['type'] == 'margin': | ||||
continue | ||||
elif block['type'] == 'section': | ||||
break | ||||
elif block['type'] == 'bullet': | ||||
if block['indent'] != 0: | ||||
raise error.Abort(_('indented bullet lists not supported')) | ||||
Rishabh Madan
|
r33012 | if title: | ||
lines = [l[1:].strip() for l in block['lines']] | ||||
notefragment.append(lines) | ||||
continue | ||||
else: | ||||
lines = [[l[1:].strip() for l in block['lines']]] | ||||
Gregory Szorc
|
r32778 | |||
Rishabh Madan
|
r33012 | for block in blocks[i + 1:]: | ||
if block['type'] in ('bullet', 'section'): | ||||
break | ||||
if block['type'] == 'paragraph': | ||||
lines.append(block['lines']) | ||||
notefragment.append(lines) | ||||
continue | ||||
Gregory Szorc
|
r32778 | elif block['type'] != 'paragraph': | ||
raise error.Abort(_('unexpected block type in release notes: ' | ||||
'%s') % block['type']) | ||||
Rishabh Madan
|
r33012 | if title: | ||
notefragment.append(block['lines']) | ||||
Gregory Szorc
|
r32778 | |||
Rishabh Madan
|
r33012 | return notefragment | ||
Gregory Szorc
|
r32778 | |||
currentsection = None | ||||
for i, block in enumerate(blocks): | ||||
if block['type'] != 'section': | ||||
continue | ||||
title = block['lines'][0] | ||||
# TODO the parsing around paragraphs and bullet points needs some | ||||
# work. | ||||
if block['underline'] == '=': # main section | ||||
name = sections.sectionfromtitle(title) | ||||
if not name: | ||||
raise error.Abort(_('unknown release notes section: %s') % | ||||
title) | ||||
currentsection = name | ||||
Rishabh Madan
|
r33012 | bullet_points = gatherparagraphsbullets(i) | ||
if bullet_points: | ||||
for para in bullet_points: | ||||
notes.addnontitleditem(currentsection, para) | ||||
Gregory Szorc
|
r32778 | |||
elif block['underline'] == '-': # sub-section | ||||
if title == BULLET_SECTION: | ||||
Rishabh Madan
|
r33012 | bullet_points = gatherparagraphsbullets(i) | ||
for para in bullet_points: | ||||
notes.addnontitleditem(currentsection, para) | ||||
Gregory Szorc
|
r32778 | else: | ||
Rishabh Madan
|
r33012 | paragraphs = gatherparagraphsbullets(i, True) | ||
Gregory Szorc
|
r32778 | notes.addtitleditem(currentsection, title, paragraphs) | ||
else: | ||||
raise error.Abort(_('unsupported section type for %s') % title) | ||||
return notes | ||||
def serializenotes(sections, notes): | ||||
"""Serialize release notes from parsed fragments and notes. | ||||
This function essentially takes the output of ``parsenotesfromrevisions()`` | ||||
and ``parserelnotesfile()`` and produces output combining the 2. | ||||
""" | ||||
lines = [] | ||||
for sectionname, sectiontitle in sections: | ||||
if sectionname not in notes: | ||||
continue | ||||
lines.append(sectiontitle) | ||||
lines.append('=' * len(sectiontitle)) | ||||
lines.append('') | ||||
# First pass to emit sub-sections. | ||||
for title, paragraphs in notes.titledforsection(sectionname): | ||||
lines.append(title) | ||||
lines.append('-' * len(title)) | ||||
lines.append('') | ||||
wrapper = textwrap.TextWrapper(width=78) | ||||
for i, para in enumerate(paragraphs): | ||||
if i: | ||||
lines.append('') | ||||
lines.extend(wrapper.wrap(' '.join(para))) | ||||
lines.append('') | ||||
# Second pass to emit bullet list items. | ||||
# If the section has titled and non-titled items, we can't | ||||
# simply emit the bullet list because it would appear to come | ||||
# from the last title/section. So, we emit a new sub-section | ||||
# for the non-titled items. | ||||
nontitled = notes.nontitledforsection(sectionname) | ||||
if notes.titledforsection(sectionname) and nontitled: | ||||
# TODO make configurable. | ||||
lines.append(BULLET_SECTION) | ||||
lines.append('-' * len(BULLET_SECTION)) | ||||
lines.append('') | ||||
for paragraphs in nontitled: | ||||
wrapper = textwrap.TextWrapper(initial_indent='* ', | ||||
subsequent_indent=' ', | ||||
width=78) | ||||
lines.extend(wrapper.wrap(' '.join(paragraphs[0]))) | ||||
wrapper = textwrap.TextWrapper(initial_indent=' ', | ||||
subsequent_indent=' ', | ||||
width=78) | ||||
for para in paragraphs[1:]: | ||||
lines.append('') | ||||
lines.extend(wrapper.wrap(' '.join(para))) | ||||
lines.append('') | ||||
Rishabh Madan
|
r33781 | if lines and lines[-1]: | ||
Gregory Szorc
|
r32778 | lines.append('') | ||
return '\n'.join(lines) | ||||
@command('releasenotes', | ||||
Rishabh Madan
|
r33883 | [('r', 'rev', '', _('revisions to process for release notes'), _('REV')), | ||
('c', 'check', False, _('checks for validity of admonitions (if any)'), | ||||
Rishabh Madan
|
r33942 | _('REV')), | ||
('l', 'list', False, _('list the available admonitions with their title'), | ||||
None)], | ||||
Rishabh Madan
|
r33883 | _('hg releasenotes [-r REV] [-c] FILE')) | ||
def releasenotes(ui, repo, file_=None, **opts): | ||||
Gregory Szorc
|
r32778 | """parse release notes from commit messages into an output file | ||
Given an output file and set of revisions, this command will parse commit | ||||
messages for release notes then add them to the output file. | ||||
Release notes are defined in commit messages as ReStructuredText | ||||
directives. These have the form:: | ||||
.. directive:: title | ||||
content | ||||
Each ``directive`` maps to an output section in a generated release notes | ||||
file, which itself is ReStructuredText. For example, the ``.. feature::`` | ||||
directive would map to a ``New Features`` section. | ||||
Release note directives can be either short-form or long-form. In short- | ||||
form, ``title`` is omitted and the release note is rendered as a bullet | ||||
list. In long form, a sub-section with the title ``title`` is added to the | ||||
section. | ||||
The ``FILE`` argument controls the output file to write gathered release | ||||
notes to. The format of the file is:: | ||||
Section 1 | ||||
========= | ||||
... | ||||
Section 2 | ||||
========= | ||||
... | ||||
Only sections with defined release notes are emitted. | ||||
If a section only has short-form notes, it will consist of bullet list:: | ||||
Section | ||||
======= | ||||
* Release note 1 | ||||
* Release note 2 | ||||
If a section has long-form notes, sub-sections will be emitted:: | ||||
Section | ||||
======= | ||||
Note 1 Title | ||||
------------ | ||||
Description of the first long-form note. | ||||
Note 2 Title | ||||
------------ | ||||
Description of the second long-form note. | ||||
If the ``FILE`` argument points to an existing file, that file will be | ||||
parsed for release notes having the format that would be generated by this | ||||
command. The notes from the processed commit messages will be *merged* | ||||
into this parsed set. | ||||
During release notes merging: | ||||
* Duplicate items are automatically ignored | ||||
* Items that are different are automatically ignored if the similarity is | ||||
greater than a threshold. | ||||
This means that the release notes file can be updated independently from | ||||
this command and changes should not be lost when running this command on | ||||
that file. A particular use case for this is to tweak the wording of a | ||||
release note after it has been added to the release notes file. | ||||
""" | ||||
Rishabh Madan
|
r33572 | sections = releasenotessections(ui, repo) | ||
Rishabh Madan
|
r33942 | if opts.get('list'): | ||
return _getadmonitionlist(ui, sections) | ||||
Rishabh Madan
|
r33883 | rev = opts.get('rev') | ||
Gregory Szorc
|
r32778 | revs = scmutil.revrange(repo, [rev or 'not public()']) | ||
Rishabh Madan
|
r33883 | if opts.get('check'): | ||
return checkadmonitions(ui, repo, sections.names(), revs) | ||||
Gregory Szorc
|
r32778 | incoming = parsenotesfromrevisions(repo, sections.names(), revs) | ||
try: | ||||
with open(file_, 'rb') as fh: | ||||
notes = parsereleasenotesfile(sections, fh.read()) | ||||
except IOError as e: | ||||
if e.errno != errno.ENOENT: | ||||
raise | ||||
notes = parsedreleasenotes() | ||||
notes.merge(ui, incoming) | ||||
with open(file_, 'wb') as fh: | ||||
fh.write(serializenotes(sections, notes)) | ||||
@command('debugparsereleasenotes', norepo=True) | ||||
Rishabh Madan
|
r33572 | def debugparsereleasenotes(ui, path, repo=None): | ||
Gregory Szorc
|
r32778 | """parse release notes and print resulting data structure""" | ||
if path == '-': | ||||
text = sys.stdin.read() | ||||
else: | ||||
with open(path, 'rb') as fh: | ||||
text = fh.read() | ||||
Rishabh Madan
|
r33572 | sections = releasenotessections(ui, repo) | ||
Gregory Szorc
|
r32778 | |||
notes = parsereleasenotesfile(sections, text) | ||||
for section in notes: | ||||
ui.write(_('section: %s\n') % section) | ||||
for title, paragraphs in notes.titledforsection(section): | ||||
ui.write(_(' subsection: %s\n') % title) | ||||
for para in paragraphs: | ||||
ui.write(_(' paragraph: %s\n') % ' '.join(para)) | ||||
for paragraphs in notes.nontitledforsection(section): | ||||
ui.write(_(' bullet point:\n')) | ||||
for para in paragraphs: | ||||
ui.write(_(' paragraph: %s\n') % ' '.join(para)) | ||||