de505df5 Andy McCall 1 month ago ↶
1
2
3
4
use super :: history:: { render_files, FileRows} ;
5
use super :: settings:: { csrf, require} ;
6
use super :: { filters, lookup, lookup_role, Error, Page, RepoShell, Shell} ;
7
use crate :: auth:: { Access, Role, Viewer} ;
8
use crate :: db:: { self , accounts, proposals as pdb} ;
9
use crate :: render:: { self , diff} ;
10
use crate :: repos:: proposals:: { self as pl, Projection, Source} ;
11
use crate :: repos:: view:: { self , ChangeView, ConflictView} ;
12
use crate :: repos:: RepoName;
13
use crate :: Shared;
14
use askama:: Template;
15
use axum:: extract:: { Form, Path, Query, State} ;
16
use axum:: response:: { IntoResponse, Redirect, Response} ;
17
use hub_api:: { Approval, ProposalSource, ProposalState} ;
18
use oxy_core:: hash:: ChangeHash;
19
use serde:: Deserialize;
20
use std:: collections:: { BTreeMap, HashMap} ;
21
22
23
24
# [ derive ( Template ) ]
25
# [ template ( path = " proposals.html" ) ]
26
pub struct ProposalsPage {
27
shell : Shell,
28
repo : RepoShell,
29
rows : Vec< pdb:: ProposalRow> ,
30
filter : String,
31
counts : ( usize , usize , usize ) ,
32
bookmarks : Vec< String> ,
1ac84c62 Andy McCall 1 month ago ↶
33
q : String,
34
sort : String,
35
pager : super:: Pager,
36
labels : std:: collections:: HashMap< i64 , Vec< db:: issues:: Label> > ,
de505df5 Andy McCall 1 month ago ↶
37
}
38
1ac84c62 Andy McCall 1 month ago ↶
39
impl ProposalsPage {
40
fn labels_of ( & self , id : & i64 ) -> & [ db:: issues:: Label] {
41
self .labels.get ( id) .map ( Vec :: as_slice) .unwrap_or ( & [ ] )
42
}
43
}
44
de505df5 Andy McCall 1 month ago ↶
45
# [ derive ( Deserialize ) ]
46
pub struct ListQuery {
47
state : Option< String> ,
1ac84c62 Andy McCall 1 month ago ↶
48
q : Option< String> ,
49
sort : Option< String> ,
50
page : Option< usize > ,
de505df5 Andy McCall 1 month ago ↶
51
}
52
1ac84c62 Andy McCall 1 month ago ↶
53
54
pub const SORTS : & [ ( & str , & str ) ] = & [ ( " updated" , " recently updated" ) , ( " newest" , " newest" ) , ( " oldest" , " oldest" ) ] ;
55
de505df5 Andy McCall 1 month ago ↶
56
pub async fn list (
57
State( app ) : State< Shared> ,
58
viewer : Viewer,
59
Path( (owner , name ) ) : Path< ( String, String) > ,
60
Query( q) : Query< ListQuery> ,
61
) -> Result< Page< ProposalsPage> , Error> {
62
let shell = Shell:: new( & app, & format! ( " /{owner} /{name} /proposals" ) , & viewer) ;
63
let ( repo, row, role) = lookup ( & app, & shell, & viewer, & owner, & name) .await? ;
64
let all = pdb:: list( & app.db, row.id, & [ ] ) .await.map_err ( | e | Error:: from_db( shell.clone ( ) , e) ) ? ;
65
let counts = (
66
all.iter ( ) .filter ( | p | p.is_open ( ) ) .count ( ) ,
67
all.iter ( ) .filter ( | p | p.state = = " landed" ) .count ( ) ,
68
all.iter ( ) .filter ( | p | p.state = = " closed" ) .count ( ) ,
69
) ;
70
let filter = q.state.unwrap_or_else ( | | " open" .into ( ) ) ;
1ac84c62 Andy McCall 1 month ago ↶
71
let mut labels: std:: collections:: HashMap< i64 , Vec< db:: issues:: Label> > = std:: collections:: HashMap:: new( ) ;
72
for ( id, l) in db:: issues:: labels_by_proposal( & app.db, row.id) .await.map_err ( | e | Error:: from_db( shell.clone ( ) , e) ) ? {
73
labels.entry ( id) .or_default ( ) .push ( l) ;
74
}
75
76
let qtext = q.q.clone ( ) .unwrap_or_default ( ) ;
77
let ( mut authors, mut wanted, mut targets, mut words) = ( vec! [ ] , vec! [ ] , vec! [ ] , vec! [ ] ) ;
78
for tok in qtext.split_whitespace ( ) {
79
match tok.split_once ( ' :' ) {
80
Some ( ( " author" , v) ) => authors.push ( v.to_lowercase ( ) ) ,
81
Some ( ( " label" , v) ) => wanted.push ( v.to_lowercase ( ) ) ,
82
Some ( ( " target" , v) ) => targets.push ( v.to_string ( ) ) ,
83
_ => words.push ( tok.to_lowercase ( ) ) ,
84
}
85
}
86
let mut rows: Vec< pdb:: ProposalRow> = all
de505df5 Andy McCall 1 month ago ↶
87
.into_iter ( )
88
.filter ( | p | match filter.as_str ( ) {
89
" open" => p.is_open ( ) ,
90
" landed" => p.state = = " landed" ,
91
" closed" => p.state = = " closed" ,
92
_ => true ,
93
} )
1ac84c62 Andy McCall 1 month ago ↶
94
.filter ( | p | authors.is_empty ( ) | | authors.contains ( & p.author.to_lowercase ( ) ) )
95
.filter ( | p | targets.is_empty ( ) | | targets.contains ( & p.target) )
96
.filter ( | p | wanted.iter ( ) .all ( | w | labels.get ( & p.id) .is_some_and ( | ls | ls.iter ( ) .any ( | l | l.name.to_lowercase ( ) = = * w ) ) ) )
97
.filter ( | p | words.iter ( ) .all ( | w | p.title.to_lowercase ( ) .contains ( w) | | p.body.to_lowercase ( ) .contains ( w) ) )
de505df5 Andy McCall 1 month ago ↶
98
.collect ( ) ;
1ac84c62 Andy McCall 1 month ago ↶
99
let sort = q.sort.filter ( | s | SORTS .iter ( ) .any ( | ( k , _) | k = = s ) ) .unwrap_or_else ( | | " updated" .into ( ) ) ;
100
match sort.as_str ( ) {
101
" newest" => rows.sort_by_key ( | p | std:: cmp:: Reverse( p.number) ) ,
102
" oldest" => rows.sort_by_key ( | p | p.number ) ,
103
_ => rows.sort_by_key ( | p | std:: cmp:: Reverse( p.updated_ms) ) ,
104
}
105
let pager = super :: Pager:: new( rows.len ( ) , q.page, & [ ( " state" , & filter) , ( " q" , & qtext) , ( " sort" , & sort) ] ) ;
106
let rows = pager.slice ( rows) ;
de505df5 Andy McCall 1 month ago ↶
107
let bookmarks = app
108
.repos
109
.read ( & repo, | r , _| Ok ( r.bookmarks ( ) ? .into_iter ( ) .map ( | b | b.name ) .collect:: < Vec< _ > > ( ) ) )
110
.await
111
.map_err ( | e | Error:: from_repo( shell.clone ( ) , e) ) ? ;
1ac84c62 Andy McCall 1 month ago ↶
112
Ok ( Page( ProposalsPage {
113
shell,
114
repo: RepoShell:: new( & app, & row, " proposals" ) .with_role ( role) ,
115
rows,
116
filter,
117
counts,
118
bookmarks,
119
q: qtext,
120
sort,
121
pager,
122
labels,
123
} ) )
de505df5 Andy McCall 1 month ago ↶
124
}
125
126
127
128
# [ derive ( Template ) ]
129
# [ template ( path = " proposal_new.html" ) ]
130
pub struct NewProposalPage {
131
shell : Shell,
132
repo : RepoShell,
133
bookmarks : Vec< String> ,
134
source : String,
135
target : String,
136
error : Option< String> ,
137
138
preview : Vec< ChangeView> ,
139
conflicted : bool ,
140
}
141
142
# [ derive ( Deserialize ) ]
143
pub struct NewQuery {
144
source : Option< String> ,
145
target : Option< String> ,
146
}
147
148
0b20f1ec Andy McCall 1 month ago ↶
149
150
151
de505df5 Andy McCall 1 month ago ↶
152
fn parse_source ( app : & Shared, repo : & RepoName, source : & str ) -> Option< ( Option< RepoName> , String) > {
0b20f1ec Andy McCall 1 month ago ↶
153
let plain = | b : & str | ! b.is_empty ( ) & & ! b.chars ( ) .any ( | c | c.is_control ( ) | | c = = ' :' ) ;
de505df5 Andy McCall 1 month ago ↶
154
if let Some ( ( slug, bookmark) ) = source.split_once ( ' :' ) {
155
let fork = RepoName:: parse( slug) ? ;
0b20f1ec Andy McCall 1 month ago ↶
156
if ! plain ( bookmark) {
de505df5 Andy McCall 1 month ago ↶
157
return None ;
0b20f1ec Andy McCall 1 month ago ↶
158
}
159
if fork = = * repo {
160
return Some ( ( None , bookmark.to_string ( ) ) ) ;
de505df5 Andy McCall 1 month ago ↶
161
}
0b20f1ec Andy McCall 1 month ago ↶
162
if ! app.repos.exists ( & fork) {
163
return None ;
164
}
de505df5 Andy McCall 1 month ago ↶
165
return Some ( ( Some ( fork) , bookmark.to_string ( ) ) ) ;
166
}
0b20f1ec Andy McCall 1 month ago ↶
167
plain ( source) .then ( | | ( None , source.to_string ( ) ) )
de505df5 Andy McCall 1 month ago ↶
168
}
169
170
async fn preview_for (
171
app : & Shared,
172
repo : & RepoName,
173
target : & str ,
174
source : & str ,
175
) -> Result< ( Vec< ChangeView> , bool ) , crate:: repos:: RepoError> {
176
let Some ( ( fork, bookmark) ) = parse_source ( app, repo, source) else { return Ok ( ( vec! [ ] , false ) ) } ;
177
let target = target.to_string ( ) ;
178
let fork_dir = fork.as_ref ( ) .map ( | f | app.repos.dir ( f) ) ;
179
app.repos
180
.read ( repo, move | r, cache| {
181
let projection = match & fork_dir {
182
Some ( dir) => {
183
let fork_repo = oxy_core:: repo:: Repo:: open( dir) ? ;
184
let heads = fork_repo.bookmarks ( ) ? .into_iter ( ) .find ( | b | b.name = = bookmark ) .map ( | b | b.heads ) .unwrap_or_default ( ) ;
185
pl:: project( r, cache, & target, Source:: Fork { fork: & fork_repo, heads } ) ?
186
}
187
None => {
188
let heads = r.bookmarks ( ) ? .into_iter ( ) .find ( | b | b.name = = bookmark ) .map ( | b | b.heads ) .unwrap_or_default ( ) ;
189
pl:: project( r, cache, & target, Source:: Bookmark( heads) ) ?
190
}
191
} ;
192
Ok ( ( projection.changes, projection.conflicted) )
193
} )
194
.await
195
}
196
197
pub async fn new_form (
198
State( app ) : State< Shared> ,
199
viewer : Viewer,
200
Path( (owner , name ) ) : Path< ( String, String) > ,
201
Query( q) : Query< NewQuery> ,
202
) -> Result< Page< NewProposalPage> , Error> {
203
let shell = Shell:: new( & app, & format! ( " /{owner} /{name} /proposals/new" ) , & viewer) ;
204
require ( & viewer, & shell) ? ;
205
let ( repo, row, role) = lookup ( & app, & shell, & viewer, & owner, & name) .await? ;
206
let bookmarks = app
207
.repos
208
.read ( & repo, | r , _| Ok ( r.bookmarks ( ) ? .into_iter ( ) .map ( | b | b.name ) .collect:: < Vec< _ > > ( ) ) )
209
.await
210
.map_err ( | e | Error:: from_repo( shell.clone ( ) , e) ) ? ;
211
let target = q.target.unwrap_or_else ( | | row.default_bookmark.clone ( ) ) ;
212
let source = q.source.unwrap_or_default ( ) ;
213
let ( preview, conflicted) =
214
if source.is_empty ( ) { ( vec! [ ] , false ) } else { preview_for ( & app, & repo, & target, & source) .await.unwrap_or ( ( vec! [ ] , false ) ) } ;
215
Ok ( Page( NewProposalPage {
216
shell,
217
repo: RepoShell:: new( & app, & row, " proposals" ) .with_role ( role) ,
218
bookmarks,
219
source,
220
target,
221
error: None ,
222
preview,
223
conflicted,
224
} ) )
225
}
226
227
# [ derive ( Deserialize ) ]
228
pub struct NewForm {
229
#[serde(rename = "_csrf")]
230
csrf_token : String,
231
title : String,
232
#[serde(default)]
233
body : String,
234
target : String,
235
source : String,
236
#[serde(default)]
237
draft : Option< String> ,
238
}
239
b64e2537 Andy McCall 1 month ago ↶
240
241
# [ allow ( clippy::too_many_arguments ) ]
242
pub async fn open_proposal (
243
app : & Shared,
244
repo : & RepoName,
245
row : & db:: repos:: RepoRow,
246
me : & accounts:: Account,
247
title : & str ,
248
body : & str ,
249
target : & str ,
250
source : & str ,
251
draft : bool ,
252
) -> Result< pdb:: ProposalRow, String> {
253
if title.trim ( ) .is_empty ( ) {
254
return Err ( " Give the proposal a title." .into ( ) ) ;
de505df5 Andy McCall 1 month ago ↶
255
}
b64e2537 Andy McCall 1 month ago ↶
256
let bookmarks =
257
app.repos.read ( repo, | r , _| Ok ( r.bookmarks ( ) ? .into_iter ( ) .map ( | b | b.name ) .collect:: < Vec< _ > > ( ) ) ) .await.map_err ( | e | e.to_string ( ) ) ? ;
258
if ! bookmarks.iter ( ) .any ( | b | b = = target ) {
259
return Err ( " Pick a target bookmark of this repository." .into ( ) ) ;
de505df5 Andy McCall 1 month ago ↶
260
}
b64e2537 Andy McCall 1 month ago ↶
261
let Some ( ( fork, bookmark) ) = parse_source ( app, repo, source.trim ( ) ) else {
262
return Err ( " The source is a bookmark of this repository, or owner/name:bookmark of a fork." .into ( ) ) ;
de505df5 Andy McCall 1 month ago ↶
263
} ;
0b20f1ec Andy McCall 1 month ago ↶
264
if fork.is_none ( ) & & ! bookmarks.contains ( & bookmark) {
265
return Err ( format! ( " There is no bookmark {bookmark} in this repository." ) ) ;
266
}
b64e2537 Andy McCall 1 month ago ↶
267
if fork.is_none ( ) & & bookmark = = target {
268
return Err ( " The source and the target are the same bookmark." .into ( ) ) ;
de505df5 Andy McCall 1 month ago ↶
269
}
b64e2537 Andy McCall 1 month ago ↶
270
let ( preview, _ ) = preview_for ( app, repo, target, source.trim ( ) ) .await.map_err ( | e | e.to_string ( ) ) ? ;
de505df5 Andy McCall 1 month ago ↶
271
if preview.is_empty ( ) {
b64e2537 Andy McCall 1 month ago ↶
272
return Err ( " The source adds nothing to the target." .into ( ) ) ;
de505df5 Andy McCall 1 month ago ↶
273
}
274
let source_repo_id = match & fork {
b64e2537 Andy McCall 1 month ago ↶
275
Some ( fk) => db:: repos:: get( & app.db, & fk.owner, & fk.name) .await.map_err ( | e | e.to_string ( ) ) ? .map ( | r | r.id ) ,
de505df5 Andy McCall 1 month ago ↶
276
None => None ,
277
} ;
278
let created = pdb:: create(
279
& app.db,
280
pdb:: NewProposal {
281
repo_id: row.id,
b64e2537 Andy McCall 1 month ago ↶
282
title: title.trim ( ) ,
283
body: body.trim ( ) ,
284
author_id: Some ( me.id) ,
285
author: & me.username,
286
draft,
287
target,
de505df5 Andy McCall 1 month ago ↶
288
source_kind: if fork.is_some ( ) { " fork" } else { " bookmark" } ,
289
source_repo_id,
290
source_bookmark: & bookmark,
291
source_changes: " " ,
292
} ,
293
)
294
.await
b64e2537 Andy McCall 1 month ago ↶
295
.map_err ( | e | e.to_string ( ) ) ? ;
296
pdb:: add_event( & app.db, created.id, & me.username, " opened" , " " ) .await.ok ( ) ;
297
accounts:: audit( & app.db, Some ( me) , " proposal.opened" , & format! ( " {} #{} " , row.slug ( ) , created.number) , title) .await.ok ( ) ;
18e381ea Andy McCall 1 month ago ↶
298
db:: notify:: index( & app.db, " proposal" , row.id, created.id, created.number, & created.title, & created.body) .await.ok ( ) ;
b64e2537 Andy McCall 1 month ago ↶
299
let url = format! ( " /{} /proposal/{} " , row.slug ( ) , created.number) ;
18e381ea Andy McCall 1 month ago ↶
300
crate :: notify:: deliver(
b64e2537 Andy McCall 1 month ago ↶
301
app,
18e381ea Andy McCall 1 month ago ↶
302
crate :: notify:: Event {
303
kind: " proposal.opened" ,
b64e2537 Andy McCall 1 month ago ↶
304
title: format! ( " {} #{} : {} " , row.slug ( ) , created.number, created.title) ,
305
url,
306
actor: me,
18e381ea Andy McCall 1 month ago ↶
307
repo_id: row.id,
308
thread: Some ( ( " proposal" , created.id) ) ,
b64e2537 Andy McCall 1 month ago ↶
309
mention_text: body,
18e381ea Andy McCall 1 month ago ↶
310
direct: vec! [ ] ,
311
} ,
312
)
313
.await;
b64e2537 Andy McCall 1 month ago ↶
314
crate :: hooks:: emit(
315
app,
316
row,
317
" proposal.opened" ,
318
& me.username,
319
serde_json:: json! ( { " number" : created.number, " title" : created.title, " target" : target, " source" : source } ) ,
320
)
321
.await;
322
Ok ( created)
323
}
324
325
pub async fn create (
326
State( app ) : State< Shared> ,
327
viewer : Viewer,
328
Path( (owner , name ) ) : Path< ( String, String) > ,
329
Form( f) : Form< NewForm> ,
330
) -> Result< Response, Error> {
331
let shell = Shell:: new( & app, & format! ( " /{owner} /{name} /proposals/new" ) , & viewer) ;
332
let me = require ( & viewer, & shell) ? ;
333
csrf ( & viewer, & f.csrf_token, & shell) ? ;
334
let ( repo, row, role) = lookup ( & app, & shell, & viewer, & owner, & name) .await? ;
335
match open_proposal ( & app, & repo, & row, & me.account, & f.title, & f.body, & f.target, & f.source, f.draft.is_some ( ) ) .await {
336
Ok ( created) => Ok ( Redirect:: to( & format! ( " /{owner} /{name} /proposal/{} " , created.number) ) .into_response ( ) ) ,
337
Err ( msg) => {
338
let bookmarks =
339
app.repos.read ( & repo, | r , _| Ok ( r.bookmarks ( ) ? .into_iter ( ) .map ( | b | b.name ) .collect:: < Vec< _ > > ( ) ) ) .await.unwrap_or_default ( ) ;
340
Ok ( Page( NewProposalPage {
341
shell,
342
repo: RepoShell:: new( & app, & row, " proposals" ) .with_role ( role) ,
343
bookmarks,
344
source: f.source.clone ( ) ,
345
target: f.target.clone ( ) ,
346
error: Some ( msg) ,
347
preview: vec! [ ] ,
348
conflicted: false ,
349
} )
350
.into_response ( ) )
351
}
352
}
de505df5 Andy McCall 1 month ago ↶
353
}
354
355
356
357
358
pub struct CommentView {
359
pub row : pdb:: CommentRow,
360
pub body_html : String,
361
pub line : Option< usize > ,
362
}
363
a1b98695 Andy McCall 1 month ago ↶
364
365
366
pub struct Entry {
367
pub kind : & 'static str ,
368
pub index : usize ,
369
}
370
de505df5 Andy McCall 1 month ago ↶
371
pub struct ReviewView {
372
pub row : pdb:: ReviewRow,
373
pub body_html : String,
374
pub sealed : bool ,
375
}
376
377
378
pub struct Loaded {
379
pub row : pdb:: ProposalRow,
380
pub changes : Vec< ChangeView> ,
381
pub hashes : Vec< ChangeHash> ,
382
pub heads : Vec< ChangeHash> ,
3f02e5ea Andy McCall 1 month ago ↶
383
pub target_heads : Vec< ChangeHash> ,
de505df5 Andy McCall 1 month ago ↶
384
pub state : String,
385
pub target_state : String,
386
pub conflicted : bool ,
387
pub conflicts : Vec< ConflictView> ,
388
pub files : Vec< FileRows> ,
9df855f2 Andy McCall 1 month ago ↶
389
390
pub diffs : Vec< oxy_core:: repo:: FileDiff> ,
391
pub bases : std:: collections:: BTreeMap< String, Vec< u8 > > ,
de505df5 Andy McCall 1 month ago ↶
392
393
pub atoms : HashMap< ( String, usize ) , ( String, u32 ) > ,
394
pub sealing_keys : Vec< String> ,
395
pub touched : Vec< String> ,
396
pub owners : Vec< String> ,
397
pub landed_already : bool ,
398
}
399
400
401
pub async fn load ( app : & Shared, repo : & RepoName, row : pdb:: ProposalRow) -> Result< Loaded, crate:: repos:: RepoError> {
402
let fork_dir = match ( row.source_kind.as_str ( ) , row.source_repo_id) {
403
( " fork" , Some ( id) ) => {
404
db:: repos:: get_by_id( & app.db, id) .await.ok ( ) .flatten ( ) .and_then ( | r | RepoName:: new( & r.owner, & r.name) ) .map ( | n | app.repos.dir ( & n) )
405
}
406
_ => None ,
407
} ;
408
let r2 = row.clone ( ) ;
409
let loaded = app
410
.repos
411
.read ( repo, move | r, cache| {
412
let source = match ( r2.source_kind.as_str ( ) , & fork_dir) {
413
( " changes" , _ ) => Source:: Changes( r2.source_changes ( ) .iter ( ) .filter_map ( | h | ChangeHash:: from_hex( h) ) .collect ( ) ) ,
414
( " fork" , Some ( dir) ) => {
415
let fork_repo = oxy_core:: repo:: Repo:: open( dir) ? ;
416
let heads =
417
fork_repo.bookmarks ( ) ? .into_iter ( ) .find ( | b | b.name = = r2.source_bookmark ) .map ( | b | b.heads ) .unwrap_or_default ( ) ;
418
419
let fork_state = fork_repo.load_state ( & heads) ? ;
420
let missing: Vec< ChangeHash> = fork_state.changes ( ) .map ( | ( h , _) | * h) .filter ( | h | ! r.store ( ) .has ( h) ) .collect ( ) ;
421
if ! missing.is_empty ( ) {
422
r.import_bundle ( & fork_repo.bundle_for ( & missing) ? , r.current_policy ( ) ? .is_some ( ) ) ? ;
423
}
424
Source:: Changes( heads)
425
}
426
_ => {
427
let heads = r.bookmarks ( ) ? .into_iter ( ) .find ( | b | b.name = = r2.source_bookmark ) .map ( | b | b.heads ) .unwrap_or_default ( ) ;
428
Source:: Bookmark( heads)
429
}
430
} ;
431
let p: Projection = pl:: project( r, cache, & r2.target, source) ? ;
432
let target_loaded = cache.load ( r, & p.target_heads) ? ;
433
let files = render_files ( & p.files, & p.bases) ;
434
let mut atoms = HashMap:: new( ) ;
435
for f in & files {
436
for l in & f.lines {
437
if let Some ( n) = l.new_no {
438
if let Some ( ( h, i) ) = pl:: atom_at( & p.loaded, & f.path, n as usize ) {
439
atoms.insert ( ( f.path.clone ( ) , n as usize ) , ( h.to_hex ( ) , i) ) ;
440
}
441
}
442
}
443
}
444
let touched: Vec< String> = p.files.iter ( ) .map ( | f | f.path.clone ( ) ) .collect ( ) ;
445
let owners = pl:: owners_for( & target_loaded, & touched) ;
446
let sealing = pl:: sealing_keys( r, & p.hashes) ? .into_iter ( ) .map ( | k | k.to_hex ( ) ) .collect ( ) ;
447
let landed_already = ! p.hashes.is_empty ( ) & & p.hashes.iter ( ) .all ( | h | target_loaded.state.contains ( h) ) ;
0b20f1ec Andy McCall 1 month ago ↶
448
449
450
let ( changes, hashes) = if r2.state = = " landed" & & p.hashes.is_empty ( ) {
451
let hashes: Vec< ChangeHash> = r2.projected_changes ( ) .iter ( ) .filter_map ( | h | ChangeHash:: from_hex( h) ) .collect ( ) ;
452
let labels = HashMap:: new( ) ;
453
let changes = hashes
454
.iter ( )
455
.filter_map ( | h | Some ( view:: change_view_pub( r, h, & r.store ( ) .get_change_meta ( h) .ok ( ) ? , false , & labels) ) )
456
.collect ( ) ;
457
( changes, hashes)
458
} else {
459
( p.changes, p.hashes)
460
} ;
de505df5 Andy McCall 1 month ago ↶
461
Ok ( Loaded {
462
row: r2,
0b20f1ec Andy McCall 1 month ago ↶
463
changes,
464
hashes,
de505df5 Andy McCall 1 month ago ↶
465
heads: p.heads,
3f02e5ea Andy McCall 1 month ago ↶
466
target_heads: p.target_heads.clone ( ) ,
de505df5 Andy McCall 1 month ago ↶
467
state: p.state.to_hex ( ) ,
468
target_state: p.target_state.to_hex ( ) ,
469
conflicted: p.conflicted,
470
conflicts: p.conflicts,
471
files,
9df855f2 Andy McCall 1 month ago ↶
472
diffs: p.files.clone ( ) ,
473
bases: p.bases.clone ( ) ,
de505df5 Andy McCall 1 month ago ↶
474
atoms,
475
sealing_keys: sealing,
476
touched,
477
owners,
478
landed_already,
479
} )
480
} )
481
.await? ;
0b20f1ec Andy McCall 1 month ago ↶
482
483
484
485
if ! loaded.row.is_open ( ) {
486
return Ok ( loaded) ;
487
}
de505df5 Andy McCall 1 month ago ↶
488
let heads_csv = loaded.heads.iter ( ) .map ( | h | h.to_hex ( ) ) .collect:: < Vec< _ > > ( ) .join ( " ," ) ;
489
let changes_csv = loaded.hashes.iter ( ) .map ( | h | h.to_hex ( ) ) .collect:: < Vec< _ > > ( ) .join ( " ," ) ;
490
pdb:: set_projection( & app.db, loaded.row.id, & loaded.state, & heads_csv, & loaded.target_state, & changes_csv, loaded.conflicted)
491
.await
492
.ok ( ) ;
493
Ok ( loaded)
494
}
495
496
497
pub struct Verdict {
498
pub reviews : Vec< ReviewView> ,
499
pub approvals : Vec< Approval> ,
500
pub blockers : Vec< String> ,
501
pub depends_on : Vec< i64 > ,
502
pub requirements : pdb:: Requirements,
503
}
504
505
pub async fn verdict ( app : & Shared, loaded : & Loaded, repo_id : i64 ) -> Result< Verdict, sqlx:: Error> {
506
let rows = pdb:: reviews( & app.db, loaded.row.id) .await? ;
507
let links = accounts:: identity_accounts( & app.db) .await? ;
508
let sealed_accounts: Vec< String> = links.iter ( ) .filter ( | ( k , _) | loaded.sealing_keys.contains ( k) ) .map ( | ( _, u ) | u.clone ( ) ) .collect ( ) ;
509
510
let mut latest: BTreeMap< String, & pdb:: ReviewRow> = BTreeMap:: new( ) ;
511
for r in & rows {
512
latest.insert ( r.author.clone ( ) , r) ;
513
}
514
let mut approvals = Vec :: new( ) ;
515
for ( who, r) in & latest {
516
if r.verdict = = " approve" {
517
let sealed = sealed_accounts.contains ( who) ;
518
let key = links.iter ( ) .find ( | ( k , u ) | u = = who & & loaded.sealing_keys.contains ( k) ) .map ( | ( k , _) | k.clone ( ) ) ;
519
approvals.push ( Approval { account: who.clone ( ) , sealed, key } ) ;
520
}
521
}
522
let reviews = rows
523
.iter ( )
524
.map ( | r | ReviewView {
525
row: r.clone ( ) ,
526
body_html: render:: markdown:: render( & r.body, " /" , " /" ) ,
527
sealed: r.verdict = = " approve" & & sealed_accounts.contains ( & r.author) ,
528
} )
529
.collect ( ) ;
530
let requirements = pdb:: requirements( & app.db, repo_id) .await? ;
531
let mut blockers = Vec :: new( ) ;
0b20f1ec Andy McCall 1 month ago ↶
532
533
if ! loaded.row.is_open ( ) {
534
return Ok ( Verdict { reviews, approvals, blockers, depends_on: vec! [ ] , requirements } ) ;
535
}
de505df5 Andy McCall 1 month ago ↶
536
if loaded.row.state = = " draft" {
537
blockers.push ( " the proposal is a draft" .into ( ) ) ;
538
}
539
if loaded.hashes.is_empty ( ) {
540
blockers.push ( " the proposal adds no changes" .into ( ) ) ;
541
}
542
let sealed_count = approvals.iter ( ) .filter ( | a | a.sealed ) .count ( ) as i64 ;
543
if sealed_count < requirements.required_approvals {
544
blockers.push ( format! ( " {sealed_count} of {} required sealed approvals" , requirements.required_approvals) ) ;
545
}
546
if requirements.require_owners {
547
let missing: Vec< & String> = loaded.owners.iter ( ) .filter ( | o | ! approvals.iter ( ) .any ( | a | a.sealed & & & a.account = = * o ) ) .collect ( ) ;
548
if ! missing.is_empty ( ) {
549
blockers.push ( format! ( " code owners still to approve: {} " , missing.iter ( ) .map ( | s | s.as_str ( ) ) .collect:: < Vec< _ > > ( ) .join ( " , " ) ) ) ;
550
}
551
}
552
if loaded.conflicted & & ! requirements.allow_conflicts {
1e23d010 Andy McCall 1 month ago ↶
553
blockers.push ( format! (
554
" landing would carry conflicts, which this repository refuses: resolve them on the source (oxy merge {} there, fix the marked files, oxy commit, push)" ,
555
loaded.row.target
556
) ) ;
de505df5 Andy McCall 1 month ago ↶
557
}
b64e2537 Andy McCall 1 month ago ↶
558
let required = db:: hooks:: required_checks( & app.db, repo_id) .await? ;
559
if ! required.is_empty ( ) {
560
let checks = db:: hooks:: checks_for_state( & app.db, repo_id, & loaded.state) .await? ;
561
let missing: Vec< String> =
562
required.iter ( ) .filter ( | n | ! checks.iter ( ) .any ( | c | & c.name = = * n & & c.status = = " passed" ) ) .cloned ( ) .collect ( ) ;
563
if ! missing.is_empty ( ) {
564
blockers.push ( format! ( " required checks not passed on the projected state: {} " , missing.join ( " , " ) ) ) ;
565
}
566
}
de505df5 Andy McCall 1 month ago ↶
567
let requesting: Vec< & String> = latest.iter ( ) .filter ( | ( _, r ) | r.verdict = = " changes" ) .map ( | ( who , _) | who ) .collect ( ) ;
568
if ! requesting.is_empty ( ) {
569
blockers.push ( format! ( " changes requested by {} " , requesting.iter ( ) .map ( | s | s.as_str ( ) ) .collect:: < Vec< _ > > ( ) .join ( " , " ) ) ) ;
570
}
571
572
let mut depends_on = Vec :: new( ) ;
573
for other in pdb:: list( & app.db, repo_id, & [ " open" , " draft" ] ) .await? {
574
if other.id = = loaded.row.id | | other.number > loaded.row.number {
575
continue ;
576
}
577
let theirs = other.projected_changes ( ) ;
578
if ! theirs.is_empty ( ) & & theirs.iter ( ) .all ( | h | loaded.hashes.iter ( ) .any ( | m | m.to_hex ( ) = = * h ) ) {
579
depends_on.push ( other.number) ;
580
}
581
}
582
if ! depends_on.is_empty ( ) {
583
blockers.push ( format! ( " depends on {} " , depends_on.iter ( ) .map ( | n | format! ( " #{n} " ) ) .collect:: < Vec< _ > > ( ) .join ( " , " ) ) ) ;
584
}
585
Ok ( Verdict { reviews, approvals, blockers, depends_on, requirements } )
586
}
587
588
# [ derive ( Template ) ]
589
# [ template ( path = " proposal.html" ) ]
590
pub struct ProposalPage {
591
shell : Shell,
592
repo : RepoShell,
593
p : pdb:: ProposalRow,
38833ed7 Andy McCall 1 month ago ↶
594
595
closes : Vec< super:: chain:: Link> ,
a1b98695 Andy McCall 1 month ago ↶
596
597
project_choices : Option< Vec< db:: projects:: Project> > ,
598
599
timeline : Vec< Entry> ,
600
601
can_triage : bool ,
de505df5 Andy McCall 1 month ago ↶
602
body_html : String,
603
tab : String,
604
changes : Vec< ChangeView> ,
605
files : Vec< FileRows> ,
9df855f2 Andy McCall 1 month ago ↶
606
diffs : Vec< oxy_core:: repo:: FileDiff> ,
607
bases : std:: collections:: BTreeMap< String, Vec< u8 > > ,
608
609
ignore_ws : bool ,
de505df5 Andy McCall 1 month ago ↶
610
conflicts : Vec< ConflictView> ,
611
conflicted : bool ,
612
comments : Vec< CommentView> ,
613
614
placed : HashMap< ( String, usize ) , Vec< usize > > ,
615
outdated : Vec< usize > ,
616
reviews : Vec< ReviewView> ,
617
approvals : Vec< Approval> ,
618
blockers : Vec< String> ,
619
depends_on : Vec< i64 > ,
620
events : Vec< pdb:: EventRow> ,
621
owners : Vec< String> ,
622
requirements : pdb:: Requirements,
623
landed_already : bool ,
624
can_edit : bool ,
4f37afcc Andy McCall 1 month ago ↶
625
626
627
deletable_source : Option< String> ,
de505df5 Andy McCall 1 month ago ↶
628
can_land : bool ,
629
hub_can_land : bool ,
630
notice : Option< String> ,
631
error : Option< String> ,
632
heads : Vec< String> ,
633
state_hash : String,
634
source_label : String,
635
api_url : String,
18e381ea Andy McCall 1 month ago ↶
636
labels : Vec< db:: issues:: Label> ,
637
all_labels : Vec< db:: issues:: Label> ,
b64e2537 Andy McCall 1 month ago ↶
638
checks : Vec< db:: hooks:: Check> ,
639
required_checks : Vec< String> ,
e1ec9fbf Andy McCall 1 month ago ↶
640
641
projects : Vec< ( db:: projects:: Project, String) > ,
cc1cd197 Andy McCall 1 month ago ↶
642
643
requested : Vec< String> ,
8e96499f Andy McCall 1 month ago ↶
644
645
reviewable : Vec< String> ,
cc1cd197 Andy McCall 1 month ago ↶
646
647
can_request : bool ,
de505df5 Andy McCall 1 month ago ↶
648
}
649
650
# [ derive ( Deserialize ) ]
651
pub struct DetailQuery {
652
tab : Option< String> ,
a1b98695 Andy McCall 1 month ago ↶
653
9df855f2 Andy McCall 1 month ago ↶
654
#[serde(default)]
a1b98695 Andy McCall 1 month ago ↶
655
notice : Option< String> ,
656
#[serde(default)]
657
error : Option< String> ,
658
#[serde(default)]
9df855f2 Andy McCall 1 month ago ↶
659
w : Option< String> ,
de505df5 Andy McCall 1 month ago ↶
660
}
661
662
663
664
# [ allow ( clippy::too_many_arguments ) ]
665
async fn detail (
666
app : & Shared,
667
shell : Shell,
668
viewer : & Viewer,
669
owner : & str ,
670
name : & str ,
671
number : i64 ,
672
tab : String,
673
notice : Option< String> ,
674
error : Option< String> ,
675
) -> Result< Page< ProposalPage> , Error> {
676
let ( repo, row, role) = lookup ( app, & shell, viewer, owner, name) .await? ;
677
let p = pdb:: get( & app.db, row.id, number)
678
.await
679
.map_err ( | e | Error:: from_db( shell.clone ( ) , e) ) ?
680
.ok_or_else ( | | Error:: not_found( shell.clone ( ) ) ) ? ;
681
let loaded = load ( app, & repo, p) .await.map_err ( | e | Error:: from_repo( shell.clone ( ) , e) ) ? ;
682
683
if loaded.landed_already & & loaded.row.is_open ( ) {
684
pdb:: set_state( & app.db, loaded.row.id, " landed" , & loaded.target_state) .await.ok ( ) ;
685
pdb:: add_event( & app.db, loaded.row.id, " " , " landed" , " every change reached the target" ) .await.ok ( ) ;
686
}
687
let p = pdb:: get( & app.db, row.id, number) .await.map_err ( | e | Error:: from_db( shell.clone ( ) , e) ) ? .unwrap_or ( loaded.row.clone ( ) ) ;
688
let v = verdict ( app, & loaded, row.id) .await.map_err ( | e | Error:: from_db( shell.clone ( ) , e) ) ? ;
689
let comment_rows = pdb:: comments( & app.db, loaded.row.id) .await.map_err ( | e | Error:: from_db( shell.clone ( ) , e) ) ? ;
690
let events = pdb:: events( & app.db, loaded.row.id) .await.map_err ( | e | Error:: from_db( shell.clone ( ) , e) ) ? ;
691
692
let mut by_atom: HashMap< ( String, String, u32 ) , usize > = HashMap:: new( ) ;
693
for ( ( path, line) , ( h, i) ) in & loaded.atoms {
694
by_atom.insert ( ( path.clone ( ) , h.clone ( ) , * i) , * line) ;
695
}
696
let mut comments = Vec :: new( ) ;
697
let mut placed: HashMap< ( String, usize ) , Vec< usize > > = HashMap:: new( ) ;
698
let mut outdated = Vec :: new( ) ;
699
for ( idx, c) in comment_rows.into_iter ( ) .enumerate ( ) {
700
let line = if c.anchor_change.is_empty ( ) {
701
None
702
} else {
703
by_atom.get ( & ( c.path.clone ( ) , c.anchor_change.clone ( ) , c.anchor_index as u32 ) ) .copied ( )
704
} ;
705
if ! c.anchor_change.is_empty ( ) {
706
match line {
707
Some ( l) => placed.entry ( ( c.path.clone ( ) , l) ) .or_default ( ) .push ( idx) ,
708
None => outdated.push ( idx) ,
709
}
710
}
711
comments.push ( CommentView { body_html: render:: markdown:: render( & c.body, " /" , " /" ) , row: c, line } ) ;
712
}
713
let mut changes = loaded.changes.clone ( ) ;
714
super :: attribute( app, & mut changes.iter_mut ( ) .collect:: < Vec< _ > > ( ) ) .await;
715
let me = viewer.account ( ) ;
716
let can_edit = me.is_some_and ( | a | Some ( a.id) = = p.author_id | | role > = Role:: Maintain) ;
4f37afcc Andy McCall 1 month ago ↶
717
let deletable_source =
718
if p.state = = " landed" & & p.source_kind = = " bookmark" & & role > = Role:: Write & & p.source_bookmark ! = row.default_bookmark {
719
let b = p.source_bookmark.clone ( ) ;
720
app.repos
721
.read ( & repo, move | r, _ | Ok ( r.view ( ) ? .bookmarks.contains_key ( & b) ) )
722
.await
723
.unwrap_or ( false )
724
.then ( | | p.source_bookmark.clone ( ) )
725
} else {
726
None
727
} ;
de505df5 Andy McCall 1 month ago ↶
728
let can_land = role > = Role:: Write & & p.is_open ( ) & & v.blockers.is_empty ( ) ;
cc1cd197 Andy McCall 1 month ago ↶
729
let can_request = p.is_open ( ) & & me.is_some_and ( | a | Some ( a.id) = = p.author_id | | role > = Role:: Write) ;
8e96499f Andy McCall 1 month ago ↶
730
let requested = pdb:: review_requests( & app.db, p.id) .await.unwrap_or_default ( ) ;
731
let mut reviewable: Vec< String> = if can_request {
732
accounts:: assignable( & app.db, row.id, & row.owner) .await.unwrap_or_default ( ) .into_iter ( ) .filter ( | u | * u ! = p.author) .collect ( )
733
} else {
734
Vec :: new( )
735
} ;
736
for r in & requested {
737
if ! reviewable.iter ( ) .any ( | x | x.eq_ignore_ascii_case ( r) ) {
738
reviewable.push ( r.clone ( ) ) ;
739
}
740
}
14530604 Andy McCall 1 month ago ↶
741
let delegated = match viewer.account ( ) {
742
Some ( a) => super :: edit:: delegated_identity( app, a.id) .await.is_some ( ) ,
743
None => false ,
744
} ;
745
let hub_can_land = delegated | | app.repos.read ( & repo, | r , _| Ok ( r.current_policy ( ) ? .is_none ( ) ) ) .await.unwrap_or ( false ) ;
de505df5 Andy McCall 1 month ago ↶
746
let source_label = match p.source_kind.as_str ( ) {
747
" fork" => format! (
748
" fork {} " ,
749
db:: repos:: get_by_id( & app.db, p.source_repo_id.unwrap_or ( 0 ) )
750
.await
751
.ok ( )
752
.flatten ( )
753
.map ( | r | format! ( " {} :{} " , r.slug ( ) , p.source_bookmark) )
754
.unwrap_or_default ( )
755
) ,
756
" changes" => format! ( " {} changes" , p.source_changes ( ) .len ( ) ) ,
757
_ => p.source_bookmark.clone ( ) ,
758
} ;
e1ec9fbf Andy McCall 1 month ago ↶
759
let projects = super :: projects:: placements_for( app, viewer, & row, p.number) .await;
38833ed7 Andy McCall 1 month ago ↶
760
let texts: Vec< String> = std:: iter:: once( p.body.clone ( ) ) .chain ( changes.iter ( ) .map ( | c | format! ( " {} \n {} " , c.title, c.body) ) ) .collect ( ) ;
761
let closes = super :: chain:: issues_closed_by( app, & row, & texts.iter ( ) .map ( String :: as_str) .collect:: < Vec< _ > > ( ) ) .await;
a1b98695 Andy McCall 1 month ago ↶
762
let project_choices = super :: projects:: project_choices( app, viewer, & row, & projects) .await;
763
let mut dated: Vec< ( i64 , Entry) > = Vec :: new( ) ;
764
for ( index, c) in comments.iter ( ) .enumerate ( ) {
765
if c.row.anchor_change.is_empty ( ) {
766
dated.push ( ( c.row.created_ms, Entry { kind: " comment" , index } ) ) ;
767
}
768
}
769
for ( index, r) in v.reviews.iter ( ) .enumerate ( ) {
770
dated.push ( ( r.row.created_ms, Entry { kind: " review" , index } ) ) ;
771
}
772
for ( index, e) in events.iter ( ) .enumerate ( ) {
773
dated.push ( ( e.at_ms, Entry { kind: " event" , index } ) ) ;
774
}
775
dated.sort_by_key ( | ( at , _) | * at) ;
776
let timeline: Vec< Entry> = dated.into_iter ( ) .map ( | ( _, e ) | e ) .collect ( ) ;
de505df5 Andy McCall 1 month ago ↶
777
Ok ( Page( ProposalPage {
a1b98695 Andy McCall 1 month ago ↶
778
can_triage: role > = Role:: Triage,
779
timeline,
38833ed7 Andy McCall 1 month ago ↶
780
closes,
a1b98695 Andy McCall 1 month ago ↶
781
project_choices,
e1ec9fbf Andy McCall 1 month ago ↶
782
projects,
de505df5 Andy McCall 1 month ago ↶
783
shell,
784
repo: RepoShell:: new( app, & row, " proposals" ) .with_role ( role) ,
c79a1111 Andy McCall 1 month ago ↶
785
body_html: super :: issues:: markdown( app, & row, & row.default_bookmark, & p.body) .await,
de505df5 Andy McCall 1 month ago ↶
786
p,
787
tab,
788
changes,
789
files: loaded.files,
9df855f2 Andy McCall 1 month ago ↶
790
diffs: loaded.diffs,
791
bases: loaded.bases,
792
ignore_ws: false ,
de505df5 Andy McCall 1 month ago ↶
793
conflicts: loaded.conflicts,
794
conflicted: loaded.conflicted,
795
comments,
796
placed,
797
outdated,
798
reviews: v.reviews,
799
approvals: v.approvals,
800
blockers: v.blockers,
801
depends_on: v.depends_on,
802
events,
803
owners: loaded.owners,
804
requirements: v.requirements,
805
landed_already: loaded.landed_already,
806
can_edit,
4f37afcc Andy McCall 1 month ago ↶
807
deletable_source,
de505df5 Andy McCall 1 month ago ↶
808
can_land,
809
hub_can_land,
810
notice,
811
error,
812
heads: loaded.heads.iter ( ) .map ( | h | h.to_hex ( ) ) .collect ( ) ,
b64e2537 Andy McCall 1 month ago ↶
813
state_hash: loaded.state.clone ( ) ,
de505df5 Andy McCall 1 month ago ↶
814
source_label,
815
api_url: format! ( " {} /api/v1/repos/{owner} /{name} /proposals/{number} " , app.config.base_url ( ) ) ,
18e381ea Andy McCall 1 month ago ↶
816
labels: db:: issues:: proposal_labels( & app.db, loaded.row.id) .await.unwrap_or_default ( ) ,
817
all_labels: db:: issues:: labels( & app.db, row.id) .await.unwrap_or_default ( ) ,
b64e2537 Andy McCall 1 month ago ↶
818
checks: db:: hooks:: checks_for_state( & app.db, row.id, & loaded.state) .await.unwrap_or_default ( ) ,
819
required_checks: db:: hooks:: required_checks( & app.db, row.id) .await.unwrap_or_default ( ) ,
8e96499f Andy McCall 1 month ago ↶
820
requested,
821
reviewable,
cc1cd197 Andy McCall 1 month ago ↶
822
can_request,
de505df5 Andy McCall 1 month ago ↶
823
} ) )
824
}
825
a1b98695 Andy McCall 1 month ago ↶
826
827
828
fn back ( owner : & str , name : & str , number : i64 , tab : & str , notice : Option< String> , error : Option< String> ) -> Response {
829
let mut url = format! ( " /{owner} /{name} /proposal/{number} ?tab={tab} " ) ;
830
if let Some ( n) = notice {
831
url.push_str ( & format! ( " ¬ice={} " , super :: settings:: urlencode( & n) ) ) ;
832
}
833
if let Some ( e) = error {
834
url.push_str ( & format! ( " &error={} " , super :: settings:: urlencode( & e) ) ) ;
835
}
836
Redirect:: to( & url) .into_response ( )
837
}
838
de505df5 Andy McCall 1 month ago ↶
839
pub async fn show (
840
State( app ) : State< Shared> ,
841
viewer : Viewer,
842
Path( (owner , name , number ) ) : Path< ( String, String, i64 ) > ,
843
Query( q) : Query< DetailQuery> ,
844
) -> Result< Page< ProposalPage> , Error> {
845
let shell = Shell:: new( & app, & format! ( " /{owner} /{name} /proposal/{number} " ) , & viewer) ;
a1b98695 Andy McCall 1 month ago ↶
846
let mut page =
847
detail ( & app, shell, & viewer, & owner, & name, number, q.tab.unwrap_or_else ( | | " conversation" .into ( ) ) , q.notice, q.error) .await? ;
9df855f2 Andy McCall 1 month ago ↶
848
if q.w.as_deref ( ) = = Some ( " 1" ) {
849
850
page.0. files =
851
super :: history:: render_files_with( & page.0. diffs, & page.0. bases, crate :: render:: diff:: Options { ignore_whitespace: true } ) ;
852
page.0. ignore_ws = true ;
853
}
854
Ok ( page)
de505df5 Andy McCall 1 month ago ↶
855
}
856
857
858
859
# [ derive ( Deserialize ) ]
860
pub struct CommentForm {
861
#[serde(rename = "_csrf")]
862
csrf_token : String,
863
body : String,
864
#[serde(default)]
865
path : String,
866
#[serde(default)]
867
line : Option< usize > ,
868
}
869
870
pub async fn comment (
871
State( app ) : State< Shared> ,
872
viewer : Viewer,
873
Path( (owner , name , number ) ) : Path< ( String, String, i64 ) > ,
874
Form( f) : Form< CommentForm> ,
a1b98695 Andy McCall 1 month ago ↶
875
) -> Result< Response, Error> {
de505df5 Andy McCall 1 month ago ↶
876
let shell = Shell:: new( & app, & format! ( " /{owner} /{name} /proposal/{number} " ) , & viewer) ;
877
let me = require ( & viewer, & shell) ? ;
878
csrf ( & viewer, & f.csrf_token, & shell) ? ;
879
let ( repo, row, _ ) = lookup ( & app, & shell, & viewer, & owner, & name) .await? ;
880
let p = pdb:: get( & app.db, row.id, number)
881
.await
882
.map_err ( | e | Error:: from_db( shell.clone ( ) , e) ) ?
883
.ok_or_else ( | | Error:: not_found( shell.clone ( ) ) ) ? ;
884
if f.body.trim ( ) .is_empty ( ) {
a1b98695 Andy McCall 1 month ago ↶
885
return Ok ( back ( & owner, & name, number, " conversation" , None , Some ( " An empty comment says nothing." .into ( ) ) ) ) ;
de505df5 Andy McCall 1 month ago ↶
886
}
887
let anchor = match ( f.path.trim ( ) , f.line) {
888
( " " , _ ) | ( _ , None ) => None ,
889
( path, Some ( line) ) => {
890
let loaded = load ( & app, & repo, p.clone ( ) ) .await.map_err ( | e | Error:: from_repo( shell.clone ( ) , e) ) ? ;
891
loaded.atoms.get ( & ( path.to_string ( ) , line) ) .cloned ( )
892
}
893
} ;
894
let tab = if anchor.is_some ( ) { " files" } else { " conversation" } ;
895
pdb:: add_comment(
896
& app.db,
897
p.id,
898
Some ( me.account.id) ,
899
& me.account.username,
900
f.body.trim ( ) ,
901
f.path.trim ( ) ,
902
anchor.as_ref ( ) .map ( | ( h , i ) | ( h.as_str ( ) , * i as i64 ) ) ,
903
)
904
.await
905
.map_err ( | e | Error:: from_db( shell.clone ( ) , e) ) ? ;
a1b98695 Andy McCall 1 month ago ↶
906
Ok ( back ( & owner, & name, number, tab, None , None ) )
de505df5 Andy McCall 1 month ago ↶
907
}
908
909
# [ derive ( Deserialize ) ]
910
pub struct ReviewForm {
911
#[serde(rename = "_csrf")]
912
csrf_token : String,
913
verdict : String,
914
#[serde(default)]
915
body : String,
916
}
917
918
pub async fn review (
919
State( app ) : State< Shared> ,
920
viewer : Viewer,
921
Path( (owner , name , number ) ) : Path< ( String, String, i64 ) > ,
922
Form( f) : Form< ReviewForm> ,
a1b98695 Andy McCall 1 month ago ↶
923
) -> Result< Response, Error> {
de505df5 Andy McCall 1 month ago ↶
924
let shell = Shell:: new( & app, & format! ( " /{owner} /{name} /proposal/{number} " ) , & viewer) ;
925
let me = require ( & viewer, & shell) ? ;
926
csrf ( & viewer, & f.csrf_token, & shell) ? ;
927
let ( _ , row, _ ) = lookup ( & app, & shell, & viewer, & owner, & name) .await? ;
928
let p = pdb:: get( & app.db, row.id, number)
929
.await
930
.map_err ( | e | Error:: from_db( shell.clone ( ) , e) ) ?
931
.ok_or_else ( | | Error:: not_found( shell.clone ( ) ) ) ? ;
932
if ! matches! ( f.verdict.as_str ( ) , " comment" | " changes" | " approve" ) {
933
return Err ( Error:: bad( shell, " a review is comment, changes or approve" ) ) ;
934
}
935
if f.verdict = = " approve" & & Some ( me.account.id) = = p.author_id {
a1b98695 Andy McCall 1 month ago ↶
936
return Ok ( back ( & owner, & name, number, " conversation" , None , Some ( " You cannot approve your own proposal." .into ( ) ) ) ) ;
de505df5 Andy McCall 1 month ago ↶
937
}
938
pdb:: add_review( & app.db, p.id, me.account.id, & me.account.username, & f.verdict, f.body.trim ( ) )
939
.await
940
.map_err ( | e | Error:: from_db( shell.clone ( ) , e) ) ? ;
941
pdb:: add_event( & app.db, p.id, & me.account.username, & format! ( " review.{} " , f.verdict) , " " ) .await.ok ( ) ;
cc1cd197 Andy McCall 1 month ago ↶
942
reviewed ( & app, & row, & p, & me.account, & f.verdict) .await;
b64e2537 Andy McCall 1 month ago ↶
943
crate :: hooks:: emit( & app, & row, " review" , & me.account.username, serde_json:: json! ( { " number" : number, " verdict" : f.verdict } ) ) .await;
de505df5 Andy McCall 1 month ago ↶
944
let notice = if f.verdict = = " approve" {
945
Some ( format! (
946
" Approval recorded. It counts once you seal the changes: oxy seal --proposal {} /{owner} /{name} /proposal/{number} " ,
947
app.config.base_url ( )
948
) )
949
} else {
950
None
951
} ;
a1b98695 Andy McCall 1 month ago ↶
952
Ok ( back ( & owner, & name, number, " conversation" , notice, None ) )
de505df5 Andy McCall 1 month ago ↶
953
}
954
955
# [ derive ( Deserialize ) ]
956
pub struct EditForm {
957
#[serde(rename = "_csrf")]
958
csrf_token : String,
959
title : String,
960
#[serde(default)]
961
body : String,
962
target : String,
963
}
964
965
pub async fn edit (
966
State( app ) : State< Shared> ,
967
viewer : Viewer,
968
Path( (owner , name , number ) ) : Path< ( String, String, i64 ) > ,
969
Form( f) : Form< EditForm> ,
a1b98695 Andy McCall 1 month ago ↶
970
) -> Result< Response, Error> {
de505df5 Andy McCall 1 month ago ↶
971
let shell = Shell:: new( & app, & format! ( " /{owner} /{name} /proposal/{number} " ) , & viewer) ;
972
let me = require ( & viewer, & shell) ? ;
973
csrf ( & viewer, & f.csrf_token, & shell) ? ;
974
let ( _ , row, role) = lookup ( & app, & shell, & viewer, & owner, & name) .await? ;
975
let p = pdb:: get( & app.db, row.id, number)
976
.await
977
.map_err ( | e | Error:: from_db( shell.clone ( ) , e) ) ?
978
.ok_or_else ( | | Error:: not_found( shell.clone ( ) ) ) ? ;
979
if Some ( me.account.id) ! = p.author_id & & role < Role:: Maintain {
980
return Err ( Error:: not_found( shell) ) ;
981
}
982
let target = if crate :: repos:: names:: valid_segment( f.target.trim ( ) ) { f.target.trim ( ) } else { & p.target } ;
983
pdb:: edit( & app.db, p.id, f.title.trim ( ) , f.body.trim ( ) , target) .await.map_err ( | e | Error:: from_db( shell.clone ( ) , e) ) ? ;
984
pdb:: add_event( & app.db, p.id, & me.account.username, " edited" , " " ) .await.ok ( ) ;
a1b98695 Andy McCall 1 month ago ↶
985
Ok ( back ( & owner, & name, number, " conversation" , Some ( " Saved." .into ( ) ) , None ) )
de505df5 Andy McCall 1 month ago ↶
986
}
987
988
# [ derive ( Deserialize ) ]
989
pub struct StateForm {
990
#[serde(rename = "_csrf")]
991
csrf_token : String,
992
action : String,
993
}
994
995
pub async fn state (
996
State( app ) : State< Shared> ,
997
viewer : Viewer,
998
Path( (owner , name , number ) ) : Path< ( String, String, i64 ) > ,
999
Form( f) : Form< StateForm> ,
a1b98695 Andy McCall 1 month ago ↶
1000
) -> Result< Response, Error> {
de505df5 Andy McCall 1 month ago ↶
1001
let shell = Shell:: new( & app, & format! ( " /{owner} /{name} /proposal/{number} " ) , & viewer) ;
1002
let me = require ( & viewer, & shell) ? ;
1003
csrf ( & viewer, & f.csrf_token, & shell) ? ;
1004
let ( _ , row, role) = lookup ( & app, & shell, & viewer, & owner, & name) .await? ;
1005
let p = pdb:: get( & app.db, row.id, number)
1006
.await
1007
.map_err ( | e | Error:: from_db( shell.clone ( ) , e) ) ?
1008
.ok_or_else ( | | Error:: not_found( shell.clone ( ) ) ) ? ;
1009
if Some ( me.account.id) ! = p.author_id & & role < Role:: Triage {
1010
return Err ( Error:: not_found( shell) ) ;
1011
}
1012
let new_state = match ( f.action.as_str ( ) , p.state.as_str ( ) ) {
1013
( " close" , " open" | " draft" ) => " closed" ,
1014
( " reopen" , " closed" ) => " open" ,
1015
( " ready" , " draft" ) => " open" ,
1016
( " draft" , " open" ) => " draft" ,
1017
_ => return Err ( Error:: bad( shell, " that transition is not possible" ) ) ,
1018
} ;
1019
pdb:: set_state( & app.db, p.id, new_state, " " ) .await.map_err ( | e | Error:: from_db( shell.clone ( ) , e) ) ? ;
1020
pdb:: add_event( & app.db, p.id, & me.account.username, new_state, " " ) .await.ok ( ) ;
b64e2537 Andy McCall 1 month ago ↶
1021
crate :: hooks:: emit( & app, & row, & format! ( " proposal.{new_state} " ) , & me.account.username, serde_json:: json! ( { " number" : number } ) ) .await;
a1b98695 Andy McCall 1 month ago ↶
1022
Ok ( back ( & owner, & name, number, " conversation" , None , None ) )
de505df5 Andy McCall 1 month ago ↶
1023
}
1024
1025
pub async fn resolve_comment (
1026
State( app ) : State< Shared> ,
1027
viewer : Viewer,
1028
Path( (owner , name , number , id ) ) : Path< ( String, String, i64 , i64 ) > ,
1029
Form( f) : Form< super:: session:: CsrfForm> ,
a1b98695 Andy McCall 1 month ago ↶
1030
) -> Result< Response, Error> {
de505df5 Andy McCall 1 month ago ↶
1031
let shell = Shell:: new( & app, & format! ( " /{owner} /{name} /proposal/{number} " ) , & viewer) ;
1032
require ( & viewer, & shell) ? ;
1033
csrf ( & viewer, & f.csrf, & shell) ? ;
1034
let ( _ , row, _ ) = lookup ( & app, & shell, & viewer, & owner, & name) .await? ;
1035
let p = pdb:: get( & app.db, row.id, number)
1036
.await
1037
.map_err ( | e | Error:: from_db( shell.clone ( ) , e) ) ?
1038
.ok_or_else ( | | Error:: not_found( shell.clone ( ) ) ) ? ;
1039
let current: Option< bool > = sqlx:: query_scalar( " SELECT resolved FROM proposal_comments WHERE id = ? AND proposal_id = ?" )
1040
.bind ( id)
1041
.bind ( p.id)
1042
.fetch_optional ( & app.db)
1043
.await
1044
.map_err ( | e | Error:: from_db( shell.clone ( ) , e) ) ? ;
1045
if let Some ( r) = current {
1046
pdb:: set_comment_resolved( & app.db, id, ! r) .await.map_err ( | e | Error:: from_db( shell.clone ( ) , e) ) ? ;
1047
}
a1b98695 Andy McCall 1 month ago ↶
1048
Ok ( back ( & owner, & name, number, " files" , None , None ) )
de505df5 Andy McCall 1 month ago ↶
1049
}
1050
1051
1052
1053
pub async fn land (
1054
State( app ) : State< Shared> ,
1055
viewer : Viewer,
1056
Path( (owner , name , number ) ) : Path< ( String, String, i64 ) > ,
1057
Form( f) : Form< super:: session:: CsrfForm> ,
a1b98695 Andy McCall 1 month ago ↶
1058
) -> Result< Response, Error> {
de505df5 Andy McCall 1 month ago ↶
1059
let shell = Shell:: new( & app, & format! ( " /{owner} /{name} /proposal/{number} " ) , & viewer) ;
1060
let me = require ( & viewer, & shell) ? ;
1061
csrf ( & viewer, & f.csrf, & shell) ? ;
1062
let ( repo, row, _ ) = lookup_role ( & app, & shell, & viewer, & owner, & name, Access:: Write) .await? ;
0b20f1ec Andy McCall 1 month ago ↶
1063
match land_as ( & app, & repo, & row, & me.account, number, " web" ) .await {
a1b98695 Andy McCall 1 month ago ↶
1064
Ok ( _ ) => Ok ( back ( & owner, & name, number, " conversation" , Some ( " Landed." .into ( ) ) , None ) ) ,
1f1eb06a Andy McCall 1 month ago ↶
1065
Err ( Landing:: NotFound) => Err ( Error:: not_found( shell) ) ,
a1b98695 Andy McCall 1 month ago ↶
1066
Err ( Landing:: Refused( m) ) => Ok ( back ( & owner, & name, number, " conversation" , None , Some ( m) ) ) ,
1f1eb06a Andy McCall 1 month ago ↶
1067
}
1068
}
1069
1070
1071
pub enum Landing {
1072
NotFound,
1073
Refused( String ) ,
1074
}
1075
1076
1077
1078
0b20f1ec Andy McCall 1 month ago ↶
1079
1080
1f1eb06a Andy McCall 1 month ago ↶
1081
pub async fn land_as (
1082
app : & Shared,
1083
repo : & RepoName,
1084
row : & db:: repos:: RepoRow,
1085
me : & accounts:: Account,
1086
number : i64 ,
0b20f1ec Andy McCall 1 month ago ↶
1087
via : & str ,
1f1eb06a Andy McCall 1 month ago ↶
1088
) -> Result< String, Landing> {
1089
let ( owner, name) = ( row.owner.clone ( ) , row.name.clone ( ) ) ;
1090
let refused = | e : String| Landing:: Refused( e) ;
1091
let p = pdb:: get( & app.db, row.id, number) .await.map_err ( | e | refused ( e.to_string ( ) ) ) ? .ok_or ( Landing:: NotFound) ? ;
1092
let loaded = load ( app, repo, p.clone ( ) ) .await.map_err ( | e | refused ( e.to_string ( ) ) ) ? ;
1093
let v = verdict ( app, & loaded, row.id) .await.map_err ( | e | refused ( e.to_string ( ) ) ) ? ;
1094
if ! p.is_open ( ) {
1095
return Err ( refused ( format! ( " Cannot land: the proposal is {} " , p.state) ) ) ;
1096
}
1097
if ! v.blockers.is_empty ( ) {
1098
return Err ( refused ( format! ( " Cannot land: {} " , v.blockers.join ( " ; " ) ) ) ) ;
de505df5 Andy McCall 1 month ago ↶
1099
}
1100
let target = p.target.clone ( ) ;
1101
let heads = loaded.heads.clone ( ) ;
14530604 Andy McCall 1 month ago ↶
1102
1103
1104
1105
1f1eb06a Andy McCall 1 month ago ↶
1106
let result = match super :: edit:: signer_for( app, me.id) .await {
1107
Ok ( signer) => crate :: repos:: edits:: move_bookmark( & app.repos, repo, target.clone ( ) , heads.clone ( ) , signer) .await.map ( | _| pl:: state_ref( & heads) .to_hex ( ) ) ,
14530604 Andy McCall 1 month ago ↶
1108
Err ( why) => {
1109
app.repos
1f1eb06a Andy McCall 1 month ago ↶
1110
.write ( repo, move | r, _ | {
14530604 Andy McCall 1 month ago ↶
1111
if r.current_policy ( ) ? .is_some ( ) {
1112
return Err ( crate :: repos:: RepoError:: Oxy( oxy_core:: repo:: RepoError:: Msg( format! (
1113
" this repository has a trust policy, so the landing must be signed by a key it authorises. Land from a client with oxy proposal land, or delegate to the hub. {why} "
1114
) ) ) ) ;
1115
}
1116
r.bookmark_set ( & target, & heads) ? ;
1117
Ok ( pl:: state_ref( & heads) .to_hex ( ) )
1118
} )
1119
.await
1120
}
1121
} ;
1f1eb06a Andy McCall 1 month ago ↶
1122
let state = result.map_err ( | e | refused ( e.to_string ( ) ) ) ? ;
0b20f1ec Andy McCall 1 month ago ↶
1123
pdb:: set_landed( & app.db, p.id, & state, & me.username, via) .await.map_err ( | e | refused ( e.to_string ( ) ) ) ? ;
1124
let how = if v.approvals.is_empty ( ) { format! ( " {via} , with no approval" ) } else { via.to_string ( ) } ;
1125
pdb:: add_event( & app.db, p.id, & me.username, " landed" , & how) .await.ok ( ) ;
1f1eb06a Andy McCall 1 month ago ↶
1126
crate :: hooks:: emit(
1127
app,
1128
row,
1129
" proposal.landed" ,
1130
& me.username,
1131
serde_json:: json! ( { " number" : number, " target" : p.target, " state" : state } ) ,
1132
)
1133
.await;
1134
crate :: notify:: deliver(
1135
app,
1136
crate :: notify:: Event {
1137
kind: " proposal.landed" ,
1138
title: format! ( " {owner} /{name} #{number} : {} landed" , p.title) ,
1139
url: format! ( " /{owner} /{name} /proposal/{number} " ) ,
1140
actor: me,
1141
repo_id: row.id,
1142
thread: Some ( ( " proposal" , p.id) ) ,
1143
mention_text: " " ,
1144
direct: p.author_id.into_iter ( ) .collect ( ) ,
1145
} ,
1146
)
1147
.await;
1148
if p.target = = row.default_bookmark {
1149
let messages: Vec< ( String, String) > = loaded.changes.iter ( ) .map ( | c | ( c.hash.clone ( ) , format! ( " {} \n {} " , c.title, c.body) ) ) .collect ( ) ;
1150
super :: issues:: close_from_messages( app, row, & messages) .await;
de505df5 Andy McCall 1 month ago ↶
1151
}
0b20f1ec Andy McCall 1 month ago ↶
1152
accounts:: audit( & app.db, Some ( me) , " proposal.landed" , & format! ( " {owner} /{name} #{number} " ) , & how) .await.ok ( ) ;
1f1eb06a Andy McCall 1 month ago ↶
1153
db:: repos:: record_push(
1154
& app.db,
1155
row.id,
1156
Some ( ( me.id, me.username.as_str ( ) ) ) ,
1157
& [ db:: repos:: PushOutcome {
1158
reference: p.target.clone ( ) ,
1159
ok: true ,
0b20f1ec Andy McCall 1 month ago ↶
1160
reason: format! ( " landed #{number} ({how} )" ) ,
1f1eb06a Andy McCall 1 month ago ↶
1161
old_state: loaded.target_state.clone ( ) ,
1162
new_state: state.clone ( ) ,
1163
heads: loaded.heads.len ( ) as i64 ,
1164
changes: loaded.hashes.len ( ) as i64 ,
1165
} ] ,
1166
)
1167
.await
1168
.ok ( ) ;
1169
Ok ( state)
de505df5 Andy McCall 1 month ago ↶
1170
}
1171
18e381ea Andy McCall 1 month ago ↶
1172
# [ derive ( Deserialize ) ]
1173
pub struct LabelsForm {
1174
#[serde(rename = "_csrf")]
1175
csrf_token : String,
1176
#[serde(default)]
1177
label : Vec< i64 > ,
1178
}
1179
1180
pub async fn set_labels (
1181
State( app ) : State< Shared> ,
1182
viewer : Viewer,
1183
Path( (owner , name , number ) ) : Path< ( String, String, i64 ) > ,
1184
axum_extra:: extract:: Form( f) : axum_extra:: extract:: Form< LabelsForm> ,
a1b98695 Andy McCall 1 month ago ↶
1185
) -> Result< Response, Error> {
18e381ea Andy McCall 1 month ago ↶
1186
let shell = Shell:: new( & app, & format! ( " /{owner} /{name} /proposal/{number} " ) , & viewer) ;
1187
require ( & viewer, & shell) ? ;
1188
csrf ( & viewer, & f.csrf_token, & shell) ? ;
1189
let ( _ , row, role) = lookup ( & app, & shell, & viewer, & owner, & name) .await? ;
1190
if role < Role:: Triage {
1191
return Err ( Error:: not_found( shell) ) ;
1192
}
1193
let p = pdb:: get( & app.db, row.id, number)
1194
.await
1195
.map_err ( | e | Error:: from_db( shell.clone ( ) , e) ) ?
1196
.ok_or_else ( | | Error:: not_found( shell.clone ( ) ) ) ? ;
1197
for l in db:: issues:: labels( & app.db, row.id) .await.map_err ( | e | Error:: from_db( shell.clone ( ) , e) ) ? {
1198
db:: issues:: set_proposal_label( & app.db, p.id, l.id, f.label.contains ( & l.id) ) .await.map_err ( | e | Error:: from_db( shell.clone ( ) , e) ) ? ;
1199
}
a1b98695 Andy McCall 1 month ago ↶
1200
Ok ( back ( & owner, & name, number, " conversation" , None , None ) )
18e381ea Andy McCall 1 month ago ↶
1201
}
1202
b64e2537 Andy McCall 1 month ago ↶
1203
1204
1205
pub async fn events (
1206
State( app ) : State< Shared> ,
1207
viewer : Viewer,
1208
Path( (owner , name , number ) ) : Path< ( String, String, i64 ) > ,
1209
) -> Result< axum:: response:: Sse< impl futures:: Stream< Item = Result< axum:: response:: sse:: Event, std:: convert:: Infallible> > > , Error> {
1210
use futures:: StreamExt;
1211
let shell = Shell:: new( & app, & format! ( " /{owner} /{name} /proposal/{number} " ) , & viewer) ;
1212
let ( _ , row, _ ) = lookup ( & app, & shell, & viewer, & owner, & name) .await? ;
1213
let rx = app.events.proposals.subscribe ( ) ;
1214
let repo_id = row.id;
1215
let stream = tokio_stream:: wrappers:: BroadcastStream:: new( rx) .filter_map ( move | item| async move {
1216
match item {
1217
Ok ( ( id, ev) ) if id = = repo_id => {
1218
Some ( Ok ( axum:: response:: sse:: Event:: default( ) .event ( ev.kind.clone ( ) ) .data ( serde_json:: to_string( & ev) .unwrap_or_default ( ) ) ) )
1219
}
1220
_ => None ,
1221
}
1222
} ) ;
1223
Ok ( axum:: response:: Sse:: new( stream) .keep_alive ( axum:: response:: sse:: KeepAlive:: default( ) ) )
1224
}
1225
de505df5 Andy McCall 1 month ago ↶
1226
1227
1228
pub async fn to_api ( app : & Shared, owner : & str , name : & str , loaded : & Loaded, v : & Verdict, source_url : Option< String> ) -> hub_api:: Proposal {
1229
let p = & loaded.row;
1230
hub_api:: Proposal {
1231
number: p.number,
1232
title: p.title.clone ( ) ,
1233
body: p.body.clone ( ) ,
1234
author: p.author.clone ( ) ,
1235
state: ProposalState:: parse( & p.state) .unwrap_or ( ProposalState:: Open) ,
1236
target: p.target.clone ( ) ,
1237
source: match p.source_kind.as_str ( ) {
1238
" fork" => {
1239
let ( o, n) = source_url
1240
.as_deref ( )
1241
.and_then ( | u | u.rsplit_once ( ' /' ) )
1242
.and_then ( | ( b , n ) | b.rsplit_once ( ' /' ) .map ( | ( _, o ) | ( o.to_string ( ) , n.to_string ( ) ) ) )
1243
.unwrap_or_default ( ) ;
1244
ProposalSource:: Fork { owner: o, name: n, bookmark: p.source_bookmark.clone ( ) }
1245
}
1246
" changes" => ProposalSource:: Changes { hashes: p.source_changes ( ) } ,
1247
_ => ProposalSource:: Bookmark { bookmark: p.source_bookmark.clone ( ) } ,
1248
} ,
1249
changes: loaded
1250
.changes
1251
.iter ( )
1252
.map ( | c | hub_api:: ChangeSummary {
1253
hash: c.hash.clone ( ) ,
1254
message: if c.body.is_empty ( ) { c.title.clone ( ) } else { format! ( " {} \n \n {} " , c.title, c.body) } ,
1255
author: hub_api:: Author {
1256
name: c.author.name.clone ( ) ,
1257
email: c.author.email.clone ( ) ,
1258
key: c.author.key.clone ( ) ,
1259
account: c.author.account.clone ( ) ,
760a7dd3 Andy McCall 1 month ago ↶
1260
note: hub_api:: Author:: note_for( & c.author.key, & c.author.account) ,
de505df5 Andy McCall 1 month ago ↶
1261
} ,
1262
when_ms: c.when_ms,
1263
deps: c.deps.clone ( ) ,
1264
head: c.head,
1265
signature: match & c.signature {
1266
view:: SigView:: Unsigned => hub_api:: Signature:: Unsigned,
1267
view:: SigView:: Valid { key, sealed, revoked, .. } => {
1268
hub_api:: Signature:: Valid { key: key.clone ( ) , sealed: * sealed, revoked: * revoked }
1269
}
1270
view:: SigView:: Invalid( why) => hub_api:: Signature:: Invalid { reason: why.clone ( ) } ,
1271
} ,
1272
} )
1273
.collect ( ) ,
1274
projected_heads: loaded.heads.iter ( ) .map ( | h | h.to_hex ( ) ) .collect ( ) ,
1275
projected_state: Some ( loaded.state.clone ( ) ) ,
1276
conflicted: loaded.conflicted,
1277
approvals: v.approvals.clone ( ) ,
1278
landable: p.is_open ( ) & & v.blockers.is_empty ( ) ,
1279
blockers: v.blockers.clone ( ) ,
1280
created_ms: p.created_ms,
1281
updated_ms: p.updated_ms,
1282
url: format! ( " {} /{owner} /{name} /proposal/{} " , app.config.base_url ( ) , p.number) ,
1283
repository_url: format! ( " {} /{owner} /{name} " , app.config.base_url ( ) ) ,
1284
source_url,
0b20f1ec Andy McCall 1 month ago ↶
1285
landed: ( p.state = = " landed" & & p.landed_ms > 0 ) .then ( | | hub_api:: Landed {
1286
by: p.landed_by.clone ( ) ,
1287
via: p.landed_via.clone ( ) ,
1288
when_ms: p.landed_ms,
1289
} ) ,
cc1cd197 Andy McCall 1 month ago ↶
1290
requested_reviewers: pdb:: review_requests( & app.db, p.id) .await.unwrap_or_default ( ) ,
2291f8cc Andy McCall 1 month ago ↶
1291
labels: db:: issues:: proposal_labels( & app.db, p.id)
1292
.await
1293
.unwrap_or_default ( )
1294
.into_iter ( )
1295
.map ( | l | hub_api:: Label { name: l.name, colour: l.colour, description: l.description } )
1296
.collect ( ) ,
de505df5 Andy McCall 1 month ago ↶
1297
}
1298
}
1299
1300
1301
pub fn key ( path : & str , line : impl filters:: Num) -> ( String, usize ) {
1302
( path.to_string ( ) , line.as_u64 ( ) as usize )
1303
}
1304
1305
# [ allow ( dead_code ) ]
1306
fn _unused ( _ : & diff:: Line, _ : & ConflictView, _ : & Role) { }
cc1cd197 Andy McCall 1 month ago ↶
1307
1308
1309
1310
1311
1312
1313
1314
pub async fn request_reviewers (
1315
app : & Shared,
1316
row : & db:: repos:: RepoRow,
1317
p : & pdb:: ProposalRow,
1318
by : & accounts:: Account,
1319
names : & [String],
1320
) -> Result< Vec< String> , String> {
1321
let mut found = Vec :: new( ) ;
1322
for name in names.iter ( ) .map ( | n | n.trim ( ) .trim_start_matches ( ' @' ) ) .filter ( | n | ! n.is_empty ( ) ) {
1323
let Some ( a) = accounts:: account_by_name( & app.db, name) .await.ok ( ) .flatten ( ) else {
1324
return Err ( format! ( " There is no account {name} ." ) ) ;
1325
} ;
1326
if Some ( a.id) = = p.author_id {
1327
return Err ( format! ( " {name} wrote this proposal, and cannot review it." ) ) ;
1328
}
1329
let role = crate :: auth:: roles:: role_for( & app.db, Some ( & a) , row) .await.ok ( ) .flatten ( ) ;
1330
if ! crate :: auth:: roles:: allows( role, crate :: auth:: Access:: Read) {
1331
return Err ( format! ( " {name} cannot read this repository." ) ) ;
1332
}
1333
found.push ( a) ;
1334
}
1335
let mut asked = Vec :: new( ) ;
1336
for a in found {
1337
if ! pdb:: request_review( & app.db, p.id, a.id, & by.username) .await.map_err ( | e | e.to_string ( ) ) ? {
1338
continue ;
1339
}
1340
pdb:: add_event( & app.db, p.id, & by.username, " review requested" , & a.username) .await.ok ( ) ;
1341
crate :: notify:: deliver(
1342
app,
1343
crate :: notify:: Event {
1344
kind: " proposal.review_requested" ,
1345
title: format! ( " {} /{} #{} : {} (review requested from {} )" , row.owner, row.name, p.number, p.title, a.username) ,
1346
url: format! ( " /{} /{} /proposal/{} " , row.owner, row.name, p.number) ,
1347
actor: by,
1348
repo_id: row.id,
1349
thread: None ,
1350
mention_text: " " ,
1351
direct: vec! [ a.id] ,
1352
} ,
1353
)
1354
.await;
1355
asked.push ( a.username) ;
1356
}
1357
Ok ( asked)
1358
}
1359
1360
1361
1362
pub async fn reviewed ( app : & Shared, row : & db:: repos:: RepoRow, p : & pdb:: ProposalRow, by : & accounts:: Account, verdict : & str ) {
1363
pdb:: unrequest_review( & app.db, p.id, by.id) .await.ok ( ) ;
1364
let what = match verdict {
1365
" approve" => " approved" ,
1366
" changes" => " changes requested" ,
1367
_ => " reviewed" ,
1368
} ;
1369
crate :: notify:: deliver(
1370
app,
1371
crate :: notify:: Event {
1372
kind: " proposal.review" ,
1373
title: format! ( " {} /{} #{} : {} ({what} by {} )" , row.owner, row.name, p.number, p.title, by.username) ,
1374
url: format! ( " /{} /{} /proposal/{} " , row.owner, row.name, p.number) ,
1375
actor: by,
1376
repo_id: row.id,
1377
thread: Some ( ( " proposal" , p.id) ) ,
1378
mention_text: " " ,
1379
direct: p.author_id.into_iter ( ) .collect ( ) ,
1380
} ,
1381
)
1382
.await;
1383
}
1384
1385
# [ derive ( Deserialize ) ]
1386
pub struct ReviewersForm {
1387
#[serde(rename = "_csrf")]
1388
csrf_token : String,
8e96499f Andy McCall 1 month ago ↶
1389
cc1cd197 Andy McCall 1 month ago ↶
1390
action : String,
8e96499f Andy McCall 1 month ago ↶
1391
#[serde(default)]
1392
username : Vec< String> ,
cc1cd197 Andy McCall 1 month ago ↶
1393
}
1394
1395
1396
1397
pub async fn reviewers (
1398
State( app ) : State< Shared> ,
1399
viewer : Viewer,
1400
Path( (owner , name , number ) ) : Path< ( String, String, i64 ) > ,
8e96499f Andy McCall 1 month ago ↶
1401
axum_extra:: extract:: Form( f) : axum_extra:: extract:: Form< ReviewersForm> ,
cc1cd197 Andy McCall 1 month ago ↶
1402
) -> Result< Response, Error> {
1403
let shell = Shell:: new( & app, & format! ( " /{owner} /{name} /proposal/{number} " ) , & viewer) ;
1404
let me = require ( & viewer, & shell) ? ;
1405
csrf ( & viewer, & f.csrf_token, & shell) ? ;
1406
let ( _ , row, role) = lookup ( & app, & shell, & viewer, & owner, & name) .await? ;
1407
let p = pdb:: get( & app.db, row.id, number)
1408
.await
1409
.map_err ( | e | Error:: from_db( shell.clone ( ) , e) ) ?
1410
.ok_or_else ( | | Error:: not_found( shell.clone ( ) ) ) ? ;
1411
if Some ( me.account.id) ! = p.author_id & & role < Role:: Write {
1412
return Err ( Error:: not_found( shell) ) ;
1413
}
8e96499f Andy McCall 1 month ago ↶
1414
let names: Vec< String> = f.username.iter ( ) .map ( | n | n.trim ( ) .trim_start_matches ( ' @' ) .to_string ( ) ) .filter ( | n | ! n.is_empty ( ) ) .collect ( ) ;
1415
let current = pdb:: review_requests( & app.db, p.id) .await.map_err ( | e | Error:: from_db( shell.clone ( ) , e) ) ? ;
1416
1417
let withdraw: Vec< String> = match f.action.as_str ( ) {
1418
" remove" => names.clone ( ) ,
1419
" set" => current.iter ( ) .filter ( | c | ! names.iter ( ) .any ( | n | n.eq_ignore_ascii_case ( c) ) ) .cloned ( ) .collect ( ) ,
1420
_ => Vec :: new( ) ,
1421
} ;
1422
for who in & withdraw {
1423
if let Some ( a) = accounts:: account_by_name( & app.db, who) .await.ok ( ) .flatten ( ) {
cc1cd197 Andy McCall 1 month ago ↶
1424
if pdb:: unrequest_review( & app.db, p.id, a.id) .await.map_err ( | e | Error:: from_db( shell.clone ( ) , e) ) ? {
1425
pdb:: add_event( & app.db, p.id, & me.account.username, " review request withdrawn" , & a.username) .await.ok ( ) ;
1426
}
1427
}
8e96499f Andy McCall 1 month ago ↶
1428
}
1429
let ask: Vec< String> = match f.action.as_str ( ) {
1430
" add" => names,
1431
" set" => names.into_iter ( ) .filter ( | n | ! current.iter ( ) .any ( | c | c.eq_ignore_ascii_case ( n) ) ) .collect ( ) ,
1432
_ => return Ok ( back ( & owner, & name, number, " conversation" , None , None ) ) ,
1433
} ;
1434
if ask.is_empty ( ) {
cc1cd197 Andy McCall 1 month ago ↶
1435
return Ok ( back ( & owner, & name, number, " conversation" , None , None ) ) ;
1436
}
8e96499f Andy McCall 1 month ago ↶
1437
match request_reviewers ( & app, & row, & p, & me.account, & ask) .await {
cc1cd197 Andy McCall 1 month ago ↶
1438
Ok ( asked) if asked.is_empty ( ) => Ok ( back ( & owner, & name, number, " conversation" , Some ( " Already asked." .into ( ) ) , None ) ) ,
1439
Ok ( asked) => Ok ( back ( & owner, & name, number, " conversation" , Some ( format! ( " Asked {} to review." , asked.join ( " , " ) ) ) , None ) ) ,
1440
Err ( e) => Ok ( back ( & owner, & name, number, " conversation" , None , Some ( e) ) ) ,
1441
}
1442
}